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

THY-42 add prettier to lint action #23

Merged
merged 2 commits into from
Feb 21, 2023
Merged

Conversation

kadenbking
Copy link
Collaborator

No description provided.

@kadenbking kadenbking self-assigned this Feb 21, 2023
@vercel
Copy link

vercel bot commented Feb 21, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
react-wordle ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 21, 2023 at 8:58PM (UTC)

@kadenbking
Copy link
Collaborator Author

The Action was failing because npm v9 did not include /bin/ in the execution PATH.
The new version of the prettier action 4.3 addressed this problem.
For more reference: creyD/prettier_action#113

@kadenbking kadenbking marked this pull request as ready for review February 21, 2023 21:00
Copy link
Contributor

@kenancasey kenancasey left a comment

Choose a reason for hiding this comment

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

This seems to be working now. Thanks for taking a lead on this. Sorry it was so annoying.

@kenancasey kenancasey merged commit 2b9fb79 into main Feb 21, 2023
@kenancasey kenancasey deleted the THY-42-patch-gitaction branch February 21, 2023 22:38
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.

2 participants