Skip to content

Conversation

@pallasite99
Copy link
Member

@shukra-in-spirit
Copy link
Contributor

shukra-in-spirit commented Jul 12, 2022

Can you add a status section on top with:

  • Waiting for review
  • Waiting for comment resolution
  • Waiting for merge
  • Draft
  • Trivial PR (nominal cosmetic/typo/whitespace changes)

So that reviewers can identify the stage of the PR

Copy link
Contributor

@shukra-in-spirit shukra-in-spirit left a comment

Choose a reason for hiding this comment

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

can you also add a small section for linking it to the issue(s) that it is resolving,

@pallasite99
Copy link
Member Author

@shukra-in-spirit sure I will.

a linked issue(s) section and the status one, cool

@pallasite99 pallasite99 force-pushed the add-pr-template branch 2 times, most recently from dcd8bb8 to dcbac16 Compare July 12, 2022 17:19
@pallasite99
Copy link
Member Author

@shukra-in-spirit made the requested changes

gives a draft PR template for each PR
Want committers to perform an assertive action of ticking the checklist for the activities finished.
@shukra-in-spirit
Copy link
Contributor

I had missed out the issues mentioned at the top of the page.

  • I added Semantic Versioning part as well.
  • And some cosmetic changes

Rest looks good to me. I am approving. Feel free to merge before the v0.2.0 release.
@haani-niyaz can you look into including this too in the release.

@pallasite99
Copy link
Member Author

Ok @shukra-in-spirit so I will squash and merge this here at this stage.

Hopefully the next PR gets this template to work with 📦

@pallasite99 pallasite99 merged commit 4b82ab8 into main Jul 13, 2022
@shukra-in-spirit
Copy link
Contributor

Keeping my fingers crossed for the same.

@pallasite99 pallasite99 deleted the add-pr-template branch July 13, 2022 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a PR template

3 participants