Skip to content

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.

@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
@kehoecj kehoecj requested review from jd4235 and jackswiney March 28, 2024 14:15
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
@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