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 #5340

Merged
merged 1 commit into from
Dec 28, 2023
Merged

update pr template #5340

merged 1 commit into from
Dec 28, 2023

Conversation

hom3mad3
Copy link
Contributor

Describe your changes
Updating our PR template for clearer communication and easier review.

At the moment, it's challenging to understand the goal of a PR and how to test it without having to ask the author. The suggested changes aim to improve the workflow for PR reviewers.

Tasks

  • PR name contains story or task reference
  • Changelog

@hom3mad3 hom3mad3 changed the title Update PR Template update pr template Nov 29, 2023
Copy link
Contributor

@philli-m philli-m left a comment

Choose a reason for hiding this comment

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

Love the idea i just added lots of notes on the how, sorry!! also happy to go with majority on this!

@hom3mad3 hom3mad3 force-pushed the mr-2023-11-update-pr-template branch from 5543b31 to 38c9e2a Compare December 21, 2023 10:03
@hom3mad3
Copy link
Contributor Author

hom3mad3 commented Dec 21, 2023

@mara @goapunk @philli-m updated pr template to have at least a human-friendly description and added the testing instructions to checklist. keeping in mind this is just a template and should be adapted to specific needs and context.

what do you think?

here is what my ideal PR looks like. I will probably use this format for frontend stuff as it lends itself well, specially with the screenshots.

Copy link
Contributor

@goapunk goapunk left a comment

Choose a reason for hiding this comment

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

Nice, looks good to me! Will leave for @m4ra to merge

Copy link
Contributor

@philli-m philli-m left a comment

Choose a reason for hiding this comment

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

Looking good, can see if additional updates need to be made as we go

@philli-m philli-m merged commit 52e1d59 into main Dec 28, 2023
2 checks passed
@philli-m philli-m deleted the mr-2023-11-update-pr-template branch December 28, 2023 12:14
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.

3 participants