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

WIP: add support for checking nginx configuration files #107

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

onlineque
Copy link
Contributor

Fixes #100

@kehoecj kehoecj added the OSS Community Contribution Contributions from the OSS Community label Jan 2, 2024
@kehoecj
Copy link
Contributor

kehoecj commented Jan 19, 2024

@onlineque Sorry for the long timeline to review this but we're ready to start taking a look at this. Can you resolve the conflicts before we start our review? Thanks!

@onlineque
Copy link
Contributor Author

No issue, man :-) Please, take a look, but first let me know if this way of handling the file masks for the nginx config files is feasible, as mentioned in the issue #100 itself. Thanks !

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 nginx.conf files
2 participants