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

update PR template #4523

Merged
merged 5 commits into from
Nov 5, 2020
Merged

update PR template #4523

merged 5 commits into from
Nov 5, 2020

Conversation

Borda
Copy link
Member

@Borda Borda commented Nov 4, 2020

What does this PR do?

Updating PR template with a simple bullet-list and link to the guidelines, discussed with @edenlightning

PR review

  • Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

@codecov
Copy link

codecov bot commented Nov 4, 2020

Codecov Report

Merging #4523 into master will increase coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #4523   +/-   ##
======================================
  Coverage      92%     93%           
======================================
  Files         116     116           
  Lines        8822    8769   -53     
======================================
- Hits         8129    8117   -12     
+ Misses        693     652   -41     

Copy link
Contributor

@tchaton tchaton left a comment

Choose a reason for hiding this comment

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

Great. Let's make sure the checklist is validated before merging any PR :)

@Borda Borda added the ready PRs ready to be merged label Nov 5, 2020
@Borda Borda merged commit 62ea461 into master Nov 5, 2020
@Borda Borda deleted the docs/PR-template branch November 5, 2020 21:05
@edenlightning edenlightning added this to the 1.0.x milestone Nov 10, 2020
rohitgr7 pushed a commit that referenced this pull request Nov 21, 2020
* update PR template

* Update .github/PULL_REQUEST_TEMPLATE.md

Co-authored-by: Roger Shieh <[email protected]>

* Apply suggestions from code review

Co-authored-by: edenlightning <[email protected]>

Co-authored-by: chaton <[email protected]>
Co-authored-by: Roger Shieh <[email protected]>
Co-authored-by: edenlightning <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation related ready PRs ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants