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

[CI] Code linting #20

Open
TheElan opened this issue Nov 20, 2021 · 2 comments
Open

[CI] Code linting #20

TheElan opened this issue Nov 20, 2021 · 2 comments

Comments

@TheElan
Copy link
Collaborator

TheElan commented Nov 20, 2021

Add plugin(s) which perform static analysis and include them in CI step along with action which will pinpoint lines in question. (E.g. stream not closed, infinite loop, etc)

@bombcar
Copy link
Contributor

bombcar commented Nov 20, 2021

Can a json validator be added to the quest json before merge? Is that a thing?

@TheElan
Copy link
Collaborator Author

TheElan commented Nov 20, 2021

Can a json validator be added to the quest json before merge? Is that a thing?

yes, there are tools for that, e.g. in python or as gradle plugins, which can be relatively easy included in CI.

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

No branches or pull requests

2 participants