-
Notifications
You must be signed in to change notification settings - Fork 230
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
Latest commit on master breaking non-pnp Yarn installs #72
Comments
Sorry, just saw that #71 is addressing this issue. Will keep this open until it is merged for other users to easily find the issue. |
#71 has merged, this issue may be able to be closed. |
Verified the changes worked on our setup. Thanks for the quick fix @jacobwgillespie! |
Hi @stramel , @jacobwgillespie I am trying to setup a Github action using
This is my publish action workflow :
Any pointers would be immensely helpful. |
Hey @ManasMaji, I took a look at the repo (I believe it's https://github.com/adobe/aem-cloud-service-source-migration) and I think the issue is that you don't have This Changesets action here uses the locally installed version of Most likely running |
Would throwing a custom error in such a case help? 🤔 |
Hi @jacobwgillespie , @Andarist Thank you so much for the quick help!! I really appreciate it! 🥇 👍 |
Commit bf4c612
Is breaking on our Yarn v1 (non-pnp) installs.
I switched the action back to use the previous commit and everything works fine again. (b3300fa)
The error I was getting is as follows:
The text was updated successfully, but these errors were encountered: