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

Propagate lexer from Gestalt Config to various locations #181

Merged
merged 4 commits into from
Apr 12, 2024

Conversation

credmond-git
Copy link
Collaborator

feat: pass the sentence lexer into the Gestalt Config, so it can be applied in all the loaders. Use the lexer to normalize the strings instead of all using a lowercase. #179
For most loaders add a new module configuration. Allows Env Vars among others to be more customizable. #151
Get the system properties from the system wrapper for easier debugging and testing.

…pplied in all the loaders. Use the lexer to normalize the strings instead of all using a lowercase. #179

For most loaders add a new module configuration. Allows Env Vars among others to be more customizable. #151
Get the system properties from the system wrapper for easier debugging and testing.
feat: ModuleConfig for Map and properties Config Loader.
test: add additional tests to show how to customize the sentence lexer.
test: add documentation to the example test to better describe what it is doing.
Copy link

codecov bot commented Apr 12, 2024

Codecov Report

Attention: Patch coverage is 94.44444% with 18 lines in your changes are missing coverage. Please review.

Project coverage is 90.08%. Comparing base (2935b76) to head (65793d8).

❗ Current head 65793d8 differs from pull request most recent head 4dca480. Consider uploading reports for the commit 4dca480 to get more accurate results

Files Patch % Lines
...g/github/gestalt/config/loader/PropertyLoader.java 66.66% 2 Missing and 3 partials ⚠️
.../github/gestalt/config/loader/MapConfigLoader.java 86.66% 0 Missing and 2 partials ⚠️
...a/org/github/gestalt/config/hocon/HoconLoader.java 88.23% 0 Missing and 2 partials ⚠️
...ava/org/github/gestalt/config/json/JsonLoader.java 88.23% 0 Missing and 2 partials ⚠️
...ava/org/github/gestalt/config/toml/TomlLoader.java 88.23% 0 Missing and 2 partials ⚠️
...ava/org/github/gestalt/config/yaml/YamlLoader.java 88.23% 0 Missing and 2 partials ⚠️
.../github/gestalt/config/builder/GestaltBuilder.java 98.41% 1 Missing ⚠️
...b/gestalt/config/loader/EnvironmentVarsLoader.java 92.85% 0 Missing and 1 partial ⚠️
...gestalt/config/source/InputStreamConfigSource.java 95.83% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #181      +/-   ##
==========================================
+ Coverage   90.01%   90.08%   +0.06%     
==========================================
  Files         197      214      +17     
  Lines        5788     6009     +221     
  Branches      681      699      +18     
==========================================
+ Hits         5210     5413     +203     
- Misses        364      368       +4     
- Partials      214      228      +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@credmond-git credmond-git merged commit 89eeaae into main Apr 12, 2024
2 checks passed
@credmond-git credmond-git deleted the feat/lexer-changes branch April 12, 2024 04:35
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
2.8% Duplication on New Code

See analysis details on SonarCloud

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.

1 participant