Skip to content

Commit f8b35f8

Browse files
authored
Merge pull request #752 from plotly/issue-pr-templates
Add issue and PR template from CONTRIBUTING.md
2 parents 7ae437b + 8b03d39 commit f8b35f8

File tree

2 files changed

+35
-0
lines changed

2 files changed

+35
-0
lines changed

.github/ISSUE_TEMPLATE.md

+20
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
Thanks for your interest in plotly.js!
2+
3+
Before opening an issue, please search for existing and closed issues. If your
4+
problem or idea is not addressed yet,
5+
[please open a new issue](https://github.com/plotly/plotly.js/issues/new).
6+
7+
Bug reports must be accompanied with a reproducible example. We recommend using
8+
[codepen](http://codepen.io/), [jsfiddle](https://jsfiddle.net/) or
9+
[jsbin](https://jsbin.com) to share your example.
10+
11+
Note that GitHub issues are reserved for bug reports and feature requests only.
12+
Implementation questions should be asked on
13+
community.plot.ly (tagged [`plotly-js`](http://community.plot.ly/c/plotly-js)) or on Stack Overflow (tagged
14+
[`plotly`](https://stackoverflow.com/questions/tagged/plotly)).
15+
16+
Comments on GitHub issues or pull requests should add content to the discussions.
17+
Approbation comments such as *+1* or *I would like this feature to be implemented as well*
18+
will be deleted by the maintainers. Please use
19+
[GitHub reactions](https://github.com/blog/2119-add-reactions-to-pull-requests-issues-and-comments)
20+
instead.

.github/PULL_REQUEST_TEMPLATE.md

+15
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
Thanks for your interest in plotly.js!
2+
3+
Developers are strongly encouraged to first make a PR to their own plotly.js
4+
fork and ask one of the maintainers to review the modifications there. Once the
5+
pull request is deemed satisfactory, the developer will be asked to make a pull
6+
request to the main plotly.js repo and may be asked to squash some commits
7+
before doing so.
8+
9+
Developers should `git rebase` their local branch off the latest `master` before
10+
opening a pull request.
11+
12+
Note that it is forbidden to force push (i.e. `git push -f`) to remote branches
13+
associated with opened pull requests. Force pushes make it hard for maintainers
14+
to keep track of updates. Therefore, if required, please
15+
`git merge master` into your PR branch instead of `git rebase master`.

0 commit comments

Comments
 (0)