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

Add support for .env config files #110 #121

Merged
merged 4 commits into from
Mar 28, 2024
Merged

Conversation

HakanVardarr
Copy link
Contributor

@HakanVardarr HakanVardarr commented Mar 27, 2024

I implemented a env validator using go-envparse package, also added a good.env and bad.env to test the env validator.

Closes #110

@HakanVardarr
Copy link
Contributor Author

I can work on it again if you didn't like my implementation.

@kehoecj kehoecj self-requested a review March 27, 2024 19:34
@kehoecj kehoecj added the OSS Community Contribution Contributions from the OSS Community label Mar 27, 2024
@kehoecj
Copy link
Collaborator

kehoecj commented Mar 27, 2024

Thanks for the contribution @HakanVardarr ! Looks like you're just missing a few lines in the test data struct to test your .env file validator.

@HakanVardarr
Copy link
Contributor Author

@kehoecj Oh I didn't see that struct :) I am going to look that right now

@kehoecj kehoecj added the pr-action-requested PR is awaiting feedback from the submitting developer label Mar 27, 2024
@ccoVeille
Copy link
Contributor

Interesting addition to the project, thanks

@kehoecj kehoecj removed the pr-action-requested PR is awaiting feedback from the submitting developer label Mar 27, 2024
Copy link
Collaborator

@kehoecj kehoecj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! One comment regarding error output.

pkg/validator/env.go Show resolved Hide resolved
@kehoecj kehoecj added pr-action-requested PR is awaiting feedback from the submitting developer waiting-on-maintainer-review PR is waiting to be reviewed and functionally tested by the maintainers and removed pr-action-requested PR is awaiting feedback from the submitting developer labels Mar 27, 2024
Copy link
Contributor

@jd4235 jd4235 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@kehoecj kehoecj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kehoecj kehoecj merged commit c2b5454 into Boeing:main Mar 28, 2024
4 checks passed
@kehoecj kehoecj removed the waiting-on-maintainer-review PR is waiting to be reviewed and functionally tested by the maintainers label Mar 28, 2024
shiina4119 pushed a commit to shiina4119/config-file-validator that referenced this pull request Aug 23, 2024
* Env file support

* Removed test

* Added test back

* Test Struct
shiina4119 pushed a commit to shiina4119/config-file-validator that referenced this pull request Oct 4, 2024
* Env file support

* Removed test

* Added test back

* Test Struct
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OSS Community Contribution Contributions from the OSS Community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for .env config files
4 participants