Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

amyleadem
Copy link
Contributor

@amyleadem amyleadem commented Apr 6, 2023

Summary

Updated the changelog entry for USWDS PR #5008 so that it warns users of a potential breaking change.

The break was originally flagged in Slack: https://gsa-tts.slack.com/archives/C3F14AHSQ/p1680208085858799

I have updated the PR description with the same information. We should also consider updating the 3.4.1 release notes.

Related issue

Closes #2051

Preview link

Preview link: Header changelog

Testing and review

  • Confirm that information accurately reflects and resolves the potential break
  • Confirm there are no issues with grammar or punctuation

Amy Leadem added 3 commits April 6, 2023 14:53
This alerts users to a possible breaking change caused by changing the hard-coded value of 50% to the default setting value of 33%
@amyleadem amyleadem requested review from mahoneycm and mejiaj April 7, 2023 15:10
@amyleadem amyleadem marked this pull request as ready for review April 7, 2023 15:14
Copy link
Contributor

@mahoneycm mahoneycm left a comment

Choose a reason for hiding this comment

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

Fix works as expected and I think the updated changelog listing looks good! 👍

One small comment about clarifying where line breaks would occur but it may be unnecessary given the setting name.

@mejiaj mejiaj merged commit 2ee85b8 into main Apr 14, 2023
@mejiaj mejiaj deleted the al-header-changelog-5008 branch April 14, 2023 15:19
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.

USWDS-Site: Add cautionary note to header changelog #5008

3 participants