-
Notifications
You must be signed in to change notification settings - Fork 158
refactor: migrate configuration system from JSON Schema to QuickType #1140
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
Merged
Merged
Changes from 30 commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
80f2736
feat: create zod schema for config
fabiovincenzi 29d4562
refactor: migrate config loader to Zod
fabiovincenzi a865d65
refactor: integrate Zod-based loader and pass config to services
fabiovincenzi 5a4e068
Merge branch 'main' into config
fabiovincenzi 0fe5e0a
chore: ignore jsonschema in unused deps check
fabiovincenzi 9758a5b
Merge branch 'config' of https://github.com/fabiovincenzi/git-proxy i…
fabiovincenzi 6663aa4
Merge branch 'main' into config
JamieSlome 29b26a3
Merge branch 'main' into config
fabiovincenzi c08da33
Merge branch 'main' into config
fabiovincenzi fa83356
Merge branch 'main' into config
fabiovincenzi 6f84652
fix: fix config schema
fabiovincenzi 7642274
feat: migrate configuration system from JSON Schema validation to Qui…
fabiovincenzi 0acdd16
chore: merge main
fabiovincenzi 795dc45
fix: fix failing tests
fabiovincenzi 0262b61
chore: remove unused deps
fabiovincenzi 1eaff43
fix: fix types checks errors
fabiovincenzi 1a0e295
feat: add banner on top of auto-generated types
fabiovincenzi 04d912e
test: add tests for proxy
fabiovincenzi 2fb1f3c
test: increase tes coverage
fabiovincenzi cec5819
test: increase tes coverage
fabiovincenzi 499570c
test: increase test coverage for generated config
fabiovincenzi ba40989
test: increase test coverage for config
fabiovincenzi 098db65
test: increase test coverage
fabiovincenzi 5f7aace
test: increase test coverage
fabiovincenzi 4a9f533
Merge branch 'main' into config
fabiovincenzi a2b8968
refactor: remove unused variables
fabiovincenzi b53f096
Merge branch 'main' into pr/fabiovincenzi/1140
fabiovincenzi b96ee9f
chore: remove unused jsonschema dependency
fabiovincenzi 0f2fde7
Merge branch 'main' into pr/fabiovincenzi/1140
fabiovincenzi 794dfae
fix: instantiate Proxy class in tests instead of using class directly
fabiovincenzi d665faf
Merge upstream main into pr/fabiovincenzi/1140
fabiovincenzi f71cfcf
feat: add ci step for generate-config-types check
fabiovincenzi 0d50786
chore: remove jsonschema
fabiovincenzi 8cb5f24
style: fun format
fabiovincenzi d1ba303
fix: add prettier formatting to generate-config-types script
fabiovincenzi 868aa13
Merge branch 'main' into config
fabiovincenzi d7cc4d7
Merge branch 'main' into config
kriswest e962932
test: add test for default config
fabiovincenzi faa3604
chore: use deprecated annotation for ssl paths
fabiovincenzi 45b8514
Merge branch 'main' into config
fabiovincenzi 9502187
chore: re generate config types
fabiovincenzi be05334
Merge branch 'config' of https://github.com/fabiovincenzi/git-proxy i…
fabiovincenzi 27fddc8
Merge branch 'main' into config
jescalada File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -51,5 +51,6 @@ | |
| "react": { | ||
| "version": "detect" | ||
| } | ||
| } | ||
| }, | ||
| "ignorePatterns": ["src/config/generated/config.ts"] | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.