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

Update PULL_REQUEST_TEMPLATE.md to be more clear #421

Merged
merged 2 commits into from
Apr 15, 2016
Merged

Conversation

jglovier
Copy link
Member

We are still getting lots of pull requests with unformatted and unhelpful descriptions. Perhaps the template content is unclear. This PR helps to address that.

Changes proposed in this pull request:

  • clarify the referenced issue number line
  • add placeholder items to proposed changes
  • add a note clarifying purpose

cc @HospitalRun/core-maintainers

@jkleinsc
Copy link
Member

@jglovier looks good, but I would slightly change the note:
"Note: pull requests without proper descriptions may simply be closed without further discussion. We appreciate your contributions, but need to know what you are offering in clearly described format. Thanks! (you can delete this text)"

@jglovier
Copy link
Member Author

@jkleinsc good call. 👍 Fixed.

@jkleinsc jkleinsc merged commit f09f9d0 into master Apr 15, 2016
@jglovier jglovier deleted the jglovier-patch-1 branch April 15, 2016 13:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants