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

Improve config merging #2935

Closed
Macroz opened this issue May 16, 2022 · 0 comments · Fixed by #2999
Closed

Improve config merging #2935

Macroz opened this issue May 16, 2022 · 0 comments · Fixed by #2999
Assignees
@Macroz
Labels
Enhancement
Projects
Rems task board
Milestone
Internal cleanup

Comments

@Macroz
Copy link
Collaborator

Macroz commented May 16, 2022 •
edited

Improve config merging from env so that it does not give a warning on startup for unrecognized config keys. Maybe we need to explicitly decide the keys to include.

See changes in #2917.
@Macroz Macroz created this issue from a note in Rems task board (Ideas) May 16, 2022
@Macroz Macroz added this to the Internal cleanup milestone May 18, 2022
@Macroz Macroz moved this from Ideas to Backlog in Rems task board Jun 14, 2022
@Macroz Macroz moved this from Backlog to In Progress in Rems task board Jun 17, 2022
@Macroz Macroz self-assigned this Jun 17, 2022
@Macroz Macroz mentioned this issue Sep 8, 2022
Merged
4 tasks
@Macroz Macroz moved this from In Progress to Done (newest on top) in Rems task board Sep 8, 2022
@jaakkocsc jaakkocsc moved this from Done (newest on top) to Accepted in Rems task board Sep 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Macroz Macroz

Labels
Enhancement
Projects
REMS Task Board
Archived in project
Rems task board
Accepted
Milestone
Internal cleanup
Development

Successfully merging a pull request may close this issue.

fix: don't complain about system prop and env config keys CSCfi/rems
1 participant
@Macroz