Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

set no generate models if allowed table value is empty in bff mode #97

Merged
merged 5 commits into from
Feb 21, 2025

Conversation

vani-rf
Copy link
Contributor

@vani-rf vani-rf commented Feb 20, 2025

[x] prevent import table when mode is bff and ALLOWED_TABLE is empty
[x] support log setup log level, and configure from config. new config :

LOG_LEVEL: trace | debug | warning | error  | info

[x] enhance default breaker enable to false

Copy link

codecov bot commented Feb 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag Coverage Δ
-race 72.30% <100.00%> (+0.54%) ⬆️
-tags "sonic avx" 72.30% <100.00%> (+0.46%) ⬆️
-tags go_json 72.30% <100.00%> (+0.48%) ⬆️
-tags nomsgpack 72.31% <100.00%> (+1.65%) ⬆️
go-1.21 72.01% <100.00%> (+0.08%) ⬆️
go-1.22 72.34% <100.00%> (?)
macos-latest 72.33% <100.00%> (?)
ubuntu-latest 72.33% <100.00%> (+0.40%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
config.go 82.53% <ø> (ø)
logger.go 100.00% <100.00%> (ø)
pkg/db/postgrest_request.go 73.68% <100.00%> (+0.23%) ⬆️
pkg/resource/load.go 100.00% <100.00%> (ø)
server.go 61.17% <100.00%> (+2.42%) ⬆️

... and 61 files with indirect coverage changes

@vani-rf vani-rf changed the title set no generate models if allowed table value is '*' in bff mode set no generate models if allowed table value is empty in bff mode Feb 21, 2025
@toopay toopay merged commit dc497cf into main Feb 21, 2025
25 checks passed
@toopay toopay deleted the feature/support-inverse-import-table branch February 21, 2025 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants