-
Notifications
You must be signed in to change notification settings - Fork 19
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
chore: update pr and issue templates
- Loading branch information
Showing
4 changed files
with
56 additions
and
85 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 |
---|---|---|
@@ -1,28 +1,46 @@ | ||
--- | ||
name: Bug report | ||
about: Create a report to help us improve | ||
title: '' | ||
labels: '' | ||
assignees: '' | ||
|
||
--- | ||
|
||
Please provide the following information and someone from @imgix/imgix-sdk-team will respond to your issue shortly. | ||
**Before you submit:** | ||
|
||
- [ ] Please read the [contributing guidelines](CONTRIBUTING.md) | ||
- [ ] Please search through the existing issues (both open AND closed) to see if your issue has been discussed before. Github issue search can be used for this: https://github.com/imgix/vue/issues?utf8=%E2%9C%93&q=is%3Aissue | ||
- [ ] Please ensure the problem has been isolated and reduced. This link explains more: http://css-tricks.com/6263-reduced-test-cases/ | ||
|
||
**Describe the bug** | ||
A clear and concise description of what the bug is. | ||
A clear and concise description of what the bug is. Please strive to reach the **root problem** of your issue to avoid the XY problem. See more: https://meta.stackexchange.com/questions/66377/what-is-the-xy-problem | ||
|
||
**To Reproduce** | ||
Steps to reproduce the behavior: | ||
A bug is a _demonstrable problem_ that is caused by the code in the repository. Thus, the contributors need a way to reproduce your issue - if we can't reproduce your issue, we can't help you! Also, please be as detailed as possible. | ||
|
||
[a link to a codesandox or repl.it; here is a link to a codesandbox with @imgix/vue installed which can be forked: https://codesandbox.io/s/vue-imgix-base-codesandbox-bhz8n] | ||
|
||
[alternatively, please provide a code example] | ||
|
||
**Expected behavior** | ||
```js | ||
// A *self-contained* demonstration of the problem follows... | ||
// This should be able to be dropped into a file with @imgix/vue installed and just work | ||
``` | ||
|
||
Steps to reproduce the behaviour: | ||
|
||
1. Go to '...' | ||
2. Click on '....' | ||
3. Scroll down to '....' | ||
4. See error | ||
|
||
**Expected behaviour** | ||
A clear and concise description of what you expected to happen. | ||
|
||
**Screenshots** | ||
If applicable, add screenshots to help explain your problem. | ||
|
||
**Information:** | ||
- imgix-php version: [e.g. 2.0.0] | ||
|
||
- @imgix/vue version: [e.g. v1.0] | ||
- browser version: [include link from [https://www.whatsmybrowser.org/](https://www.whatsmybrowser.org/) or details about the OS used and browser version] | ||
|
||
**Additional context** | ||
Add any other context about the problem here. |
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
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 |
---|---|---|
@@ -1,17 +1,12 @@ | ||
--- | ||
name: Question | ||
about: Ask a question about the project | ||
title: '' | ||
labels: '' | ||
assignees: | ||
|
||
--- | ||
|
||
Please provide the following information and someone from @imgix/imgix-sdk-team will respond to your issue shortly. | ||
|
||
**Before you submit:** | ||
|
||
- [ ] Please search through the [existing issues](https://github.com/imgix/imgix-php/issues?utf8=%E2%9C%93&q=is%3Aissue) to see if your question has already been discussed. | ||
- [ ] Please read the [contributing guidelines](CONTRIBUTING.md) | ||
- [ ] Please search through the existing issues (both open AND closed) to see if your question has already been discussed. Github issue search can be used for this: https://github.com/imgix/vue/issues?utf8=%E2%9C%93&q=is%3Aissue | ||
|
||
**Question** | ||
A clear and concise description of your question. | ||
A clear and concise description of your question |
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 |
---|---|---|
@@ -1,73 +1,36 @@ | ||
<!-- | ||
Hello, and thanks for contributing to imgix-php! 🎉🙌 | ||
Hello, and thanks for contributing 🎉🙌 | ||
Please take a second to fill out PRs with the following template! | ||
--> | ||
|
||
# Description | ||
## Description | ||
|
||
<!-- What is accomplished by this PR? If there is something potentially controversial in your PR, please take a moment to tell us about your choices. --> | ||
<!-- What is accomplished by this PR? If there is something potentially | ||
controversial in your PR, please take a moment to tell us about your choices.--> | ||
|
||
<!-- | ||
Please use the checklist that is most closely related to your PR, and delete the other checklists. --> | ||
<!-- Before this PR... --> | ||
|
||
## Checklist: Fixing typos/Doc change | ||
<!-- After this PR... --> | ||
|
||
- [ ] Each commit follows the [Conventional Commit](https://www.conventionalcommits.org/en/v1.0.0/#summary) format: e.g. `chore(readme): fixed typo`. See the end of this file for more information. | ||
<!-- Steps to test: either provide a code snippet that exhibits this change or a link to a codepen/codesandbox demo --> | ||
|
||
## Checklist: Bug Fix | ||
## Checklist | ||
|
||
- [ ] Each commit follows the [Conventional Commit](https://www.conventionalcommits.org/en/v1.0.0/#summary) format: e.g. `chore(readme): fixed typo`. See the end of this file for more information. | ||
- [ ] All existing unit tests are still passing (if applicable) | ||
- [ ] Add new passing unit tests to cover the code introduced by your PR | ||
- [ ] Update the readme | ||
- [ ] Update or add any necessary API documentation | ||
- [ ] Add some [steps](#steps-to-test) so we can test your bug fix | ||
|
||
## Checklist: New Feature | ||
|
||
- [ ] Each commit follows the [Conventional Commit](https://www.conventionalcommits.org/en/v1.0.0/#summary) format: e.g. `chore(readme): fixed typo`. See the end of this file for more information. | ||
- [ ] Any breaking changes are specified on the commit on which they are introduced with `BREAKING CHANGE` in the body of the commit. | ||
- [ ] If this is a big feature with breaking changes, consider [opening an issue](https://github.com/imgix/imgix-php/issues/new?assignees=&labels=&template=feature_request.md&title=) to discuss first. This is completely up to you, but please keep in mind that your PR might not be accepted. | ||
- [ ] Run unit tests to ensure all existing tests are still passing | ||
- [ ] Add new passing unit tests to cover the code introduced by your PR | ||
- [ ] Update the readme | ||
- [ ] Add some [steps](#steps-to-test) so we can test your cool new feature! | ||
|
||
## Steps to Test | ||
|
||
<!-- Delete this selction if you are just submitting a doc change/small fix --> | ||
|
||
<!-- A code example or a set of steps is preferred --> | ||
|
||
Related issue: [e.g. #42] | ||
|
||
Code: | ||
|
||
```php | ||
// A standalone example of what the PR solves | ||
``` | ||
|
||
<!-- A link to a codepen/codesandbox is also an option. --> | ||
|
||
<!-- | ||
<!-- Please ensure you've completed this checklist before submitting a PR. If | ||
You're not submitting a bugfix or feature, delete that part of the checklist. | ||
--> | ||
|
||
## Conventional Commit Spec | ||
<!-- For all Pull Requests --> | ||
|
||
PR titles should be in the format `<type>(<scope>): <description>`. For example: `chore(readme): fix typo` | ||
- [ ] Read the [contributing guidelines](CONTRIBUTING.md). | ||
- [ ] Each commit follows the [Conventional Commit](https://www.conventionalcommits.org/en/v1.0.0/#summary) spec format. | ||
- [ ] Update the readme (if applicable). | ||
- [ ] Update or add any necessary API documentation (if applicable) | ||
- [ ] All existing unit tests are still passing (if applicable). | ||
|
||
`type` can be any of the follow: | ||
- `feat`: a feature, or breaking change | ||
- `fix`: a bug-fix | ||
- `test`: Adding missing tests or correcting existing tests | ||
- `docs`: documentation only changes (readme, changelog, contributing guide) | ||
- `refactor`: a code change that neither fixes a bug nor adds a feature | ||
- `chore`: reoccurring tasks for project maintainability (example scopes: release, deps) | ||
- `config`: changes to tooling configurations used in the project | ||
- `build`: changes that affect the build system or external dependencies (example scopes: npm, bundler, gradle) | ||
- `ci`: changes to CI configuration files and scripts (example scopes: travis) | ||
- `perf`: a code change that improves performance | ||
- `style`: changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc) | ||
<!-- For new feature and bugfix Pull Requests--> | ||
|
||
`scope` is optional, and can be anything. | ||
`description` should be a short description of the change, written in the imperative-mood. | ||
--> | ||
- [ ] Add some [steps](#steps-to-test) so we can test your bug fix or feature (if applicable). | ||
- [ ] Add new passing unit tests to cover the code introduced by your PR (if applicable). | ||
- [ ] Any breaking changes are specified on the commit on which they are introduced with `BREAKING CHANGE` in the body of the commit. | ||
- [ ] If this is a big feature with breaking changes, consider [opening an issue][issues] to discuss first. This is completely up to you, but please keep in mind that your PR might not be accepted. |