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

Continuous integration t̶e̶s̶t̶i̶n̶g̶ linting #138

Open
2 tasks
pjf opened this issue Mar 14, 2014 · 0 comments
Open
2 tasks

Continuous integration t̶e̶s̶t̶i̶n̶g̶ linting #138

pjf opened this issue Mar 14, 2014 · 0 comments
Labels

Comments

@pjf
Copy link
Owner

pjf commented Mar 14, 2014

We already have some tools which are able to do code linting, spotting references to non-existent reactions, or single-use reaction classes, or potentially other problems.

It'd be great to:

  • Have a codebase which lints free of major errors.
  • Use travis to check to see if commits or pull requests introduce obvious errors.

The first part is the hard part. I've got lots of travis experience, and deploying it would be easy.

Alternatively, we can do something magical that records which linting errors we've seen, and insists we don't introduce new ones. But that's adding a lot of complexity, and I'd really just love to fix the errors that we already have (or mark some error types as "soft" errors that we're willing to let past).

@pjf pjf added the wishlist label Mar 14, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant