-
Notifications
You must be signed in to change notification settings - Fork 106
Contributing
Contributions are most welcome and will be accepted if the guidelines in this document are followed, so before creating a pull request, make sure you have done everything in this list:
-
Lint all files using the rules in
.eslintrc.js
, located in the root of thekarma-typescript
project. -
Create as small and descriptive functions and classes as possible.
-
Comment your code where necessary, ie where it otherwise would be hard to understand why something is done in a certain way, not what is being done. Don't litter the code with comments like
// create a new array
or// set the value
. -
If possible, add new integration tests that covers the new feature or bug fix.
-
Run all the integration tests, no tests should fail.
-
Run all the manual regression tests listed in this document.
-
Make sure the code in
example-project
andintegration-tests
compile on command line withtsc
. -
Run the tests in
example-project
usingnpm test
ornpm run dev
. -
Run all integrations tests in
integration-tests
usingnpm test
ornpm run dev
. -
Make changes to any file and make sure it gets picked up by Karma and the compiler.
-
Introduce a syntactic error in a file and check that the compiler produces a diagnostic.
-
Make sure new coverage html files has been created for both the example project and the integration tests and that they contain all the expected tests with remapped coverage.