-
Notifications
You must be signed in to change notification settings - Fork 4
Create CONTRIBUTING.md #37
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
Open
ahmetcetin
wants to merge
2
commits into
nanomosfet:master
Choose a base branch
from
ahmetcetin:patch-1
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+39
−0
Open
Changes from all commits
Commits
Show all changes
2 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,39 @@ | ||
| ## Contributing | ||
|
|
||
| Like any other open source projects, there are multiple ways to contribute to this project: | ||
|
|
||
| * As a developer, depending on your skills and experience, | ||
| * As a user who enjoys the project and wants to help. | ||
|
|
||
|
|
||
| ##### Reporting Bugs | ||
|
|
||
| If you found something broken or not working properly, feel free to create an issue in Github with as much information as possible, such as logs and how to reproduce the problem. Before opening the issue, make sure that: | ||
|
|
||
| * You have read this documentation, | ||
| * You are using the latest version of project, | ||
| * You already searched other issues to see if your problem or request was already reported. | ||
|
|
||
|
|
||
| ##### Improving the Documentation | ||
|
|
||
| You can improve this documentation by forking its repository, updating the content and sending a pull request. | ||
|
|
||
|
|
||
| #### We ❤️ Pull Requests | ||
|
|
||
| A pull request does not need to be a fix for a bug or implementing something new. Software can always be improved, legacy code removed and tests are always welcome! | ||
|
|
||
| After you make a code change, to run: | ||
|
|
||
| * Lint: npm run lint | ||
| * Test: npm run test | ||
| * Test coverage: npm run coveralls | ||
| * Build: npm run build | ||
|
|
||
| Please do not be afraid of contributing code, make sure it follows these rules: | ||
|
|
||
| * Build should pass, does not break any of the existing code in the master branch and does not cause conflicts, | ||
| * Comments are needed for confusing parts of your code only, | ||
| * An overview or context is provided as body of the Pull Request. It does not need to be too extensive. | ||
| * Tests should cover the your code, and overall test coverage should be above 90% | ||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is actually used for CI
you can do:
or add script with
"coverage" : "jest --coverage"and make this