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

feat: implement issue and pr templates #641

Merged
merged 1 commit into from Jan 8, 2018
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
33 changes: 33 additions & 0 deletions .github/ISSUE_TEMPLATE.md
@@ -0,0 +1,33 @@
## The Problem

[] πŸ› This is a bug report.
[] πŸ“ˆ This is a feature request.

<!-- Please check one of the above by placing an x in the box. -->

Briefly describe the issue you are experiencing (or the feature you want to see added to OpenZeppelin). Tell us what you were trying to do and what happened instead. **Remember, this is _not_ a place to ask for help debugging code; for that, we welcome you in the [OpenZeppelin Slack Channel](https://slack.openzeppelin.org/).**

## πŸ’» Environment

First, we need to know what your environment looks like.

- Which version of OpenZeppelin are you using?
- What network are you deploying to? testrpc? Ganache? Ropsten?
- How are you deploying your OpenZeppelin-backed contracts? truffle? Remix? Let us know!

## πŸ“ Details

Describe the problem you have been experiencing in more detail. Include as much information as you think is relevant. Keep in mind that transactions can fail for many reasons; context is key here.

## πŸ”’ Code To Reproduce Issue [ Good To Have ]

Please remember that with sample code it's easier to reproduce the bug and it's much faster to fix it.

```
insert short code snippets here
```

<!-- If your code is larger, consider linking us to a repo illustrating your issue. -->

## πŸ‘ Other Information
<!-- List any other information that is relevant to your issue. Error logs, related issues, suggestions on how to fix, Stack Overflow links, forum links, etc. -->
17 changes: 17 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
@@ -0,0 +1,17 @@
<!-- πŸŽ‰ Thank you for submitting a PR! Before submitting, please review the following checklist: -->

- [ ] πŸ“˜ I've reviewed the [OpenZeppelin Contributor Guidelines](/docs/CONTRIBUTING.md)
- [ ] βœ… I've added tests where applicable to test my new functionality.
- [ ] πŸ“– I've made sure that my contracts are well-documented.
- [ ] 🎨 I've run the JavaScript linter (`npm run lint:fix`) and fixed all issues.

<!-- **Does this close any open issues?** If so, list them here. -->

Fixes #

---

# πŸš€ Description

<!-- Describe the changes introduced in this pull request -->
<!-- Include any context necessary for understanding the PR's purpose. -->
2 changes: 2 additions & 0 deletions CONTRIBUTING.md
Expand Up @@ -80,6 +80,8 @@ git checkout -b fix/some-bug
git checkout -b remove/some-file
```

If your branch is planned to fix an open issue, postfix your branch name with the issue number like `fix/some-bug-#123`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see that this was done in this PR but Waffle didn't associate it to the corresponding issue correctly. It may be related to the lack of "fixes #639" thing mentioned above?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.


We expect pull requests to be rebased to the master branch before merging:
```sh
git remote add zep git@github.com:OpenZeppelin/zeppelin-solidity.git
Expand Down