Skip to content

Validate if all environment variables are present #2

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
Merged
merged 6 commits into from
Oct 31, 2024
Merged

Validate if all environment variables are present #2

merged 6 commits into from
Oct 31, 2024

Conversation

velo
Copy link
Collaborator

@velo velo commented Oct 30, 2024

No description provided.

@velo velo requested a review from henneberger October 30, 2024 21:00
Base automatically changed from split_classes to main October 30, 2024 21:06
@velo velo closed this Oct 30, 2024
@velo velo reopened this Oct 30, 2024
henneberger
henneberger reviewed Oct 30, 2024
View reviewed changes
henneberger
henneberger reviewed Oct 30, 2024
View reviewed changes
henneberger
henneberger reviewed Oct 30, 2024
View reviewed changes
henneberger
henneberger reviewed Oct 30, 2024
View reviewed changes
henneberger
henneberger reviewed Oct 30, 2024
View reviewed changes
henneberger
henneberger reviewed Oct 30, 2024
View reviewed changes
henneberger
henneberger reviewed Oct 30, 2024
View reviewed changes
@velo velo force-pushed the validate_env_vars branch from 2167c2d to 89666b3 Compare October 31, 2024 12:34
@velo velo requested a review from henneberger October 31, 2024 13:49
velo added 5 commits October 31, 2024 13:56
@velo
Validate if all environment variables are present
131c5e9 
Signed-off-by: Marvin Froeder <velo.br@gmail.com>
@velo
Leave variables typed
99b2063 
Signed-off-by: Marvin Froeder <velo.br@gmail.com>
@velo
Added test for EnvironmentVariablesUtils
fb1b285 
Signed-off-by: Marvin Froeder <velo.br@gmail.com>
@velo
Remove commented out code
bb42e43 
Signed-off-by: Marvin Froeder <velo.br@gmail.com>
@velo
Added simple log when all keys are present on env vars
7b263f4 
Signed-off-by: Marvin Froeder <velo.br@gmail.com>
@velo velo force-pushed the validate_env_vars branch from 7334e54 to 7b263f4 Compare October 31, 2024 16:56
@velo
Add missing license headers
8cbbdda 
Signed-off-by: Marvin Froeder <velo.br@gmail.com>
@velo velo force-pushed the validate_env_vars branch from c31c2dc to 8cbbdda Compare October 31, 2024 16:57
@velo velo merged commit 2961ab0 into main Oct 31, 2024
1 check passed
@velo velo deleted the validate_env_vars branch October 31, 2024 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@henneberger henneberger henneberger approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@velo @henneberger