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

PR Template: Add checklist item for changelog #5176

Merged
merged 5 commits into from
Mar 24, 2023
Merged
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
16 changes: 15 additions & 1 deletion .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,13 @@ _Indicate if this update is a breaking change with **one** of the following stat
This is not a breaking change.
:warning: This is potentially a breaking change.
:warning: This is a breaking change.
<!-- If applicable, explain what actions are required for the user to remediate the break. -->
<!--
Breaking changes include:
- Changes to the JavaScript API
- Changes to markup or content in our components
- Significant changes to the display of a component
If applicable, explain what actions are required for the user to remediate the break.
-->
Copy link
Member

Choose a reason for hiding this comment

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

In general, I don't think we should add a Changelog requirement to our PR template, but assure that all the information is there to add one if we need to as part of the release process. To that end, I've added a section on what makes a breaking change...


## Related issue

Expand All @@ -50,6 +56,14 @@ If no open issue exists, you can open one here:
https://github.com/uswds/uswds/issues/new/choose.
-->

## Related pull requests

_Indicate if there are other pull requests necessary to complete this issue._

<!--
Some changes to the USWDS codebase require a change to the documentation site, and need a pull request in the uswds-site repo. This could include new or updated component documentation, new or updated settings documentation, or changelog entries. Add links to any related PRs in this section. If this change requires an update to the uswds-site repo, but that PR does yet exist, just make sure to note that here.
-->
Copy link
Contributor

Choose a reason for hiding this comment

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

These new additions are great!
One small thing: it would be good to introduce line breaks in these comments to make it more readable in GitHub source. One possibility:

Suggested change
<!--
Some changes to the USWDS codebase require a change to the documentation site, and need a pull request in the uswds-site repo. This could include new or updated component documentation, new or updated settings documentation, or changelog entries. Add links to any related PRs in this section. If this change requires an update to the uswds-site repo, but that PR does yet exist, just make sure to note that here.
-->
<!--
Some changes to the USWDS codebase require a change to the documentation site,
and need a pull request in the uswds-site repo.
This could include:
1. New or updated component documentation
2. New or updated settings documentation, or
3. Changelog entries.
Add links to any related PRs in this section.
If this change requires an update to the uswds-site repo, but that PR does yet exist,
just make sure to note that here.
-->

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've added this change in 0d5d451. I've also:

  • Changed it to an unordered list to match previous lists
  • Added additional new lines for clarity
  • Moved text to new line after 80 characters
  • Condensed last paragraph


## Preview link

Preview link:
Expand Down