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.

Sign up for GitHub

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

Jump to bottom

Validate configuration properties #569

Closed
DirkMahler opened this issue Aug 1, 2024 · 0 comments · Fixed by #570
Closed

Validate configuration properties #569

DirkMahler opened this issue Aug 1, 2024 · 0 comments · Fixed by #570
Assignees
@DirkMahler
Labels
feature
Milestone
2.4

Comments

@DirkMahler
Copy link
Member

DirkMahler commented Aug 1, 2024 •
edited
Loading

Currently it is possible to use config properties files that are not evaluated (because of invalid indent in YAML files...). Therefore it would be helpful if jQA fails while loading the configuration if properties with the prefix jqassistant are detected that cannot be mapped to supported properties.
@DirkMahler DirkMahler self-assigned this Aug 2, 2024
DirkMahler added a commit that referenced this issue Aug 2, 2024
@DirkMahler
#569 activated validation of unknown properties
7fa00f0
@DirkMahler DirkMahler linked a pull request Aug 2, 2024 that will close this issue
569-validate-configuration-properties #570
Merged
DirkMahler added a commit that referenced this issue Aug 2, 2024
@DirkMahler
#569 updated ConfigurationSerializer to not export empty collections
759dc6f
DirkMahler added a commit that referenced this issue Aug 2, 2024
@DirkMahler
#569 added option to ignore jQA properties
aa6198e
DirkMahler added a commit that referenced this issue Aug 3, 2024
@DirkMahler
#569 fixed Maven ITs
dfa0ace
@DirkMahler DirkMahler added this to the 2.4 milestone Aug 5, 2024
DirkMahler added a commit that referenced this issue Aug 5, 2024
@DirkMahler
#569 do not serialize empty maps
acbe1cf
DirkMahler added a commit that referenced this issue Aug 5, 2024
@DirkMahler
#569-validate-configuration-properties (#570)
a770169
DirkMahler added a commit that referenced this issue Aug 7, 2024
@DirkMahler
#569 validate unknown properties only when debug is active
a4c9859
DirkMahler added a commit that referenced this issue Aug 7, 2024
@DirkMahler
#569 fixed ConfigurationMappingLoaderTest
7f2e4cd
DirkMahler added a commit that referenced this issue Aug 8, 2024
@DirkMahler
#569 fixed Checkstyle violation
4a13e41
DirkMahler added a commit that referenced this issue Aug 8, 2024
@DirkMahler
#569 validate configuration properties (#576)
f4a0249
DirkMahler added a commit that referenced this issue Aug 8, 2024
@DirkMahler
#569 validate configuration properties (#579)
410ba55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DirkMahler DirkMahler

Labels
feature
Projects
None yet
Milestone
2.4
Development

Successfully merging a pull request may close this issue.

569-validate-configuration-properties jQAssistant/jqassistant
1 participant
@DirkMahler