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

Added Codemods for Style Dictionary v4 #1323

Merged
merged 7 commits into from
Sep 18, 2024

Conversation

AkshitGarg24
Copy link
Contributor

@AkshitGarg24 AkshitGarg24 commented Aug 31, 2024

🚀 Feature Proposal

Building codemods for Upgrading from Style Dictionary v3 to Style Dictionary v4.
Looking forward to add these and help Style Dictionary users have a smooth migration experience.

Motivation
The motivation to create codemods is that it would streamline the process for users, and also ensure that the application stays efficient and secure while leveraging the latest advancements in Style Dictionary.

Example
These codemods would help migrate from Style Dictionary v3 to Style Dictionary v4 real quick as running :

npx codemod styledictionary/4/migration-recipe

which would include all the codemods.

@AkshitGarg24 AkshitGarg24 requested a review from a team as a code owner August 31, 2024 06:35
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-1323.d16eby4ekpss5y.amplifyapp.com

jorenbroekema
jorenbroekema previously approved these changes Sep 2, 2024
Copy link
Collaborator

@jorenbroekema jorenbroekema left a comment

Choose a reason for hiding this comment

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

Super awesome! I'm gonna create an SD v3 project tomorrow in a way where I can test all of these codemods in an actual project, but other than that consider this approved from my side

@dbanksdesign dbanksdesign merged commit c0f368b into amzn:main Sep 18, 2024
5 checks passed
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.

4 participants