Skip to content

chore: update changelog#96

Merged
georgewrmarshall merged 3 commits into
mainfrom
fix/95/update-changelog
Nov 13, 2024
Merged

chore: update changelog#96
georgewrmarshall merged 3 commits into
mainfrom
fix/95/update-changelog

Conversation

@georgewrmarshall
Copy link
Copy Markdown
Contributor

@georgewrmarshall georgewrmarshall commented Nov 9, 2024

Description

This PR updates the CHANGELOG tag links for the design-tokens package to align with the MetaMask module template’s naming convention, following the migration of this package to the metamask-design-system monorepo. The updated tag format ensures that all tag diffs are consistent with the naming scheme and are correctly linked to the new @metamask/design-tokens namespace for future releases.

Related issues

Fixes: #95

Manual testing steps

  1. Navigate to merged-packages/design-tokens within the metamask-design-system repository.
  2. Run the following command:
    ../../scripts/validate-changelog.sh @metamask/design-tokens
  3. Confirm that the script returns no output, indicating that all tag diff links follow the correct format.

Screenshots/Recordings

After

after.mov

Pre-merge author checklist

  • I've followed MetaMask Contributor Docs.
  • I've completed the PR template to the best of my ability.
  • I’ve included tests if applicable.
  • I’ve documented my code using JSDoc format if applicable.
  • I’ve applied the right labels on the PR (see labeling guidelines).

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g., run the validate-changelog.sh script to verify correct tag formatting).
  • I confirm that this PR addresses all acceptance criteria and includes necessary testing evidence.

@socket-security
Copy link
Copy Markdown

socket-security Bot commented Nov 9, 2024

No dependency changes detected. Learn more about Socket for GitHub ↗︎

👍 No dependency changes detected in pull request

@georgewrmarshall georgewrmarshall changed the title Fix/95/update changelog chore: update changelog Nov 12, 2024
@georgewrmarshall georgewrmarshall force-pushed the fix/95/update-changelog branch 2 times, most recently from 2ec3b1a to 7aabb90 Compare November 13, 2024 00:53
@georgewrmarshall georgewrmarshall marked this pull request as ready for review November 13, 2024 00:54
@georgewrmarshall georgewrmarshall requested a review from a team as a code owner November 13, 2024 00:54
brianacnguyen
brianacnguyen previously approved these changes Nov 13, 2024
- Use the script to ensure that the tags have the correct package name prefixes.

2. Create a fork of the metamask module template for testing, and push the ported tags to the test fork.
2. Create a fork of the metamask design system for testing, and push the ported tags to the test fork.
Copy link
Copy Markdown
Contributor Author

@georgewrmarshall georgewrmarshall Nov 13, 2024

Choose a reason for hiding this comment

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

fixes in migration docs

@georgewrmarshall georgewrmarshall merged commit 45f60ac into main Nov 13, 2024
@georgewrmarshall georgewrmarshall deleted the fix/95/update-changelog branch November 13, 2024 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Format design tokens changelog to align with module template

2 participants