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

Bear proposal: Gixy (nginx config static analyzer) #1724

Open
SanketDG opened this issue May 13, 2017 · 3 comments · May be fixed by #2754
Open

Bear proposal: Gixy (nginx config static analyzer) #1724

SanketDG opened this issue May 13, 2017 · 3 comments · May be fixed by #2754

Comments

@SanketDG
Copy link
Member

https://github.com/yandex/gixy

@jayvdb
Copy link
Member

jayvdb commented May 13, 2017

this was mentioned at #1638 . Any benefit in having two bears?

@underyx
Copy link
Member

underyx commented May 16, 2017

@jayvdb Yeah, I think we should have two:

  • nginx -t is best for validity checking, since it actually uses the real config parser to find any errors
  • gixy is best for recommending best practices and structure/style changes

@SanketDG SanketDG reopened this May 16, 2017
@NNeast NNeast mentioned this issue Oct 30, 2018
2 tasks
suggoitanoshi added a commit to suggoitanoshi/coala-bears that referenced this issue Nov 5, 2018
This commit adds GixyBear.

Closes coala#1724
@suggoitanoshi suggoitanoshi linked a pull request Nov 5, 2018 that will close this issue
2 tasks
suggoitanoshi added a commit to suggoitanoshi/coala-bears that referenced this issue Nov 5, 2018
This commit adds GixyBear.

Closes coala#1724
suggoitanoshi added a commit to suggoitanoshi/coala-bears that referenced this issue Nov 5, 2018
This commit adds GixyBear.

Closes coala#1724
suggoitanoshi added a commit to suggoitanoshi/coala-bears that referenced this issue Nov 5, 2018
This commit adds GixyBear.

Closes coala#1724
suggoitanoshi added a commit to suggoitanoshi/coala-bears that referenced this issue Nov 5, 2018
This commit adds GixyBear.

Closes coala#1724
@jayvdb
Copy link
Member

jayvdb commented Nov 11, 2018

This issue reserved for Google Code-in task 6713645942702080.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

5 participants