Skip to content
This repository has been archived by the owner on Aug 4, 2023. It is now read-only.

Add a PR template to the repository #131

Merged
merged 1 commit into from
Aug 3, 2021
Merged

Add a PR template to the repository #131

merged 1 commit into from
Aug 3, 2021

Conversation

dhruvkb
Copy link
Member

@dhruvkb dhruvkb commented Aug 2, 2021

Targets WordPress/openverse#35

This PR adds a PR template to the repository.

@dhruvkb dhruvkb requested a review from a team as a code owner August 2, 2021 16:50
@dhruvkb dhruvkb requested review from zackkrida and obulat August 2, 2021 16:50
Comment on lines +12 to +13
## Tests
<!-- Give steps for the reviewer to verify that this PR fixes the problem; or delete this section entirely. -->
Copy link
Member

Choose a reason for hiding this comment

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

Thinking again about this section, I consider it important so I'd prefer that contributors don't skip it.

Suggested change
## Tests
<!-- Give steps for the reviewer to verify that this PR fixes the problem; or delete this section entirely. -->
## Tests
<!-- Give steps for the reviewer to verify that this PR fixes the problem. -->

Copy link
Member Author

Choose a reason for hiding this comment

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

It's generally much easier to deduce if a PR works by seeing the issue it addresses.

  • If the PR fixes a bug, you can follow its steps to reproduce and they'll not reproduce the issue with the PR applied.
  • If the PR adds a feature, we can check if the feature is added.

This section would only be useful in certain edge case scenarios or when an issue does not exist (which generally would be the case when a maintainer makes a PR).

@dhruvkb dhruvkb merged commit 8764118 into main Aug 3, 2021
@dhruvkb dhruvkb deleted the pr_template branch August 3, 2021 09:50
@zackkrida zackkrida added the 🌟 goal: addition Addition of new feature label Aug 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🌟 goal: addition Addition of new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants