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

[V2] (Suggestion) Continuous Integration #360

Open
InzeNL opened this issue Sep 17, 2019 · 1 comment
Open

[V2] (Suggestion) Continuous Integration #360

InzeNL opened this issue Sep 17, 2019 · 1 comment

Comments

@InzeNL
Copy link

InzeNL commented Sep 17, 2019

Currently, V2 of tmi.js is being worked on. If the decision is ever made to implement continuous integration, it would be smarter to do this now, while it is still being built up, than later on, when it is "done". This prevents having to rewrite code later on to have it meet set standards.

Continuous integration demands higher code quality, if set up correctly, and reduces the risk of code breaking unexpectedly.

Before implementing this I would like to gauge what people think of this. I would also like to get a list of suggestions for CI tools.

I myself would suggest to have at least these:

  • Build success
    • TravisCI is free for open-source projects
  • Test Coverage (% TBD)
    • mocha + nyc are already implemented
  • Mutation Coverage (% TBD)
    • @stryker-mutator/core is widely used to mutate JS
  • Linters (Note: V2 is being written in TS, transpiled to JS)
    • TSLint
@StevenKowalzik
Copy link

As TSLint will be deprecated this year, eslint-typescript should be used as a replacement for linting.

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