mirror of
https://github.com/chartjs/Chart.js.git
synced 2024-10-06 12:19:08 +02:00
Clean up PR process
This commit is contained in:
parent
a5550cef6b
commit
63b14dde15
@ -25,9 +25,9 @@ Guidlines for reporting bugs:
|
|||||||
|
|
||||||
- Check the issue search to see if it has already been reported
|
- Check the issue search to see if it has already been reported
|
||||||
- Isolate the problem to a simple test case
|
- Isolate the problem to a simple test case
|
||||||
- Provide a demonstration of the problem on [jsbin](http://jsbin.com) or similar
|
- Provide a demonstration of the problem on [JS Bin](http://jsbin.com) or similar
|
||||||
|
|
||||||
Please provide any additional details associated with the bug, if it's browser of screen density specific, or only happens with a certain configuration or data.
|
Please provide any additional details associated with the bug, if it's browser or screen density specific, or only happens with a certain configuration or data.
|
||||||
|
|
||||||
|
|
||||||
Pull requests
|
Pull requests
|
||||||
@ -37,12 +37,13 @@ Clear, concise pull requests are excellent at continuing the project's community
|
|||||||
|
|
||||||
Guidlines:
|
Guidlines:
|
||||||
|
|
||||||
- Please ask before starting significant work on a pull request to check it's a change within the project scope, and isn't a duplicate effort
|
- Please create an issue first:
|
||||||
- Please make changes to the files in [`/src`](https://github.com/nnnick/Chart.js/tree/master/src), not `Chart.js` or `Chart.min.js` in the repo root directory
|
- For bugs, we can discuss the fixing approach
|
||||||
|
- For enhancements, we can discuss if it is within the project scope and avoid duplicate effort
|
||||||
|
- Please make changes to the files in [`/src`](https://github.com/nnnick/Chart.js/tree/master/src), not `Chart.js` or `Chart.min.js` in the repo root directory, this avoids merge conflicts
|
||||||
- Tabs for indentation, not spaces please
|
- Tabs for indentation, not spaces please
|
||||||
- If adding new functionality, please also update the relevant `.md` file in [`/docs`](https://github.com/nnnick/Chart.js/tree/master/docs)
|
- If adding new functionality, please also update the relevant `.md` file in [`/docs`](https://github.com/nnnick/Chart.js/tree/master/docs)
|
||||||
- Please make your commits in logical sections with clear commit messages
|
- Please make your commits in logical sections with clear commit messages
|
||||||
- Please avoid committing in the build Chart.js & Chart.min.js file, as it may cause conflicts when merging back
|
|
||||||
|
|
||||||
Joining the Project
|
Joining the Project
|
||||||
-------------
|
-------------
|
||||||
|
Loading…
Reference in New Issue
Block a user