-
Notifications
You must be signed in to change notification settings - Fork 56
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #44 from opentracing/36_adds_issue_and_pr_templates
[#36] Adds issue and PR templates.
- Loading branch information
Showing
2 changed files
with
44 additions
and
0 deletions.
There are no files selected for viewing
This file contains 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,21 @@ | ||
<!-- | ||
Welcome to the OpenTracing PHP repo! 👋🎉 | ||
- Please search for existing issues in order to ensure we don't have duplicate bugs/feature requests.) | ||
- Please be respectful and considerate of others when commenting on issues) | ||
- Please provide as much information as possible so we all understand the issue.) | ||
- Please don't ask questions here. If you have any question head to our gitter chat https://gitter.im/opentracing/opentracing-php | ||
--> | ||
|
||
|
||
## Background | ||
Something that gives context about why this is an issue. | ||
|
||
## Problem | ||
Describe the problem. If the issue is about an improvement you can skip this. If possible, include a description of the impact of the problem. | ||
|
||
## Proposal | ||
A proposal that from your POV would solve the problem or improve the existing situation. It should also include the impact. | ||
|
||
## Questions to address | ||
Questions that should be answered as outcome of this issue. |
This file contains 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,23 @@ | ||
<!-- | ||
We appreciate the effort for this pull request but before that please make sure you read the contribution guidelines given above, then fill out the blanks below. | ||
Please don't open huge pull requests and keep one pull request solving one problem. | ||
Please enter the issue number you are resolving in your PR after one of the following words [Fixes, Closes, Resolves]. This will auto-link these issues and close them when this PR is merged! | ||
e.g. | ||
Fixes #1 | ||
Closes #2 | ||
--> | ||
# Fixes # | ||
|
||
### Checklist | ||
- [ ] I have made a material change to the repo (functionality, testing, spelling, grammar) | ||
- [ ] I have read the [Contribution Guide] and my PR follows them. | ||
- [ ] I updated my branch with the master branch. | ||
- [ ] I have added tests that prove my fix is effective or that my feature works | ||
- [ ] I have added necessary documentation about the functionality in the appropriate .md file | ||
- [ ] I have added in line documentation to the code I modified | ||
|
||
### Short description of what this PR does: | ||
- | ||
- |