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

CI test, do not merge #3584

Closed
wants to merge 2 commits into from
Closed

Conversation

jenseng
Copy link
Contributor

@jenseng jenseng commented Jun 27, 2022

getting a weird GH actions failure around the yarn cache on this PR: #2027

seeing if i can repro with another commit

getting a weird GH actions failure around the yarn cache on this PR:
remix-run#2027

seeing if i can repro with another commit
@jenseng jenseng changed the base branch from main to dev June 27, 2022 20:54
@jenseng jenseng force-pushed the npm-install-cache-test branch 2 times, most recently from da0d1f4 to dab6435 Compare June 27, 2022 23:49
@jenseng
Copy link
Contributor Author

jenseng commented Jun 28, 2022

✅ Confirmed, this looks like a bug in bahmutov/npm-install... on Windows, installs fail when using bahmutov/npm-install@v1 (i.e. v1.8.16) but work when using bahmutov/[email protected]

@MichaelDeBoey
Copy link
Member

MichaelDeBoey commented Jun 28, 2022

@jenseng Could you please file an issue on the upstream repo for this so @bahmutov can look into this?

jenseng added a commit to jenseng/remix that referenced this pull request Jun 28, 2022
v1.8.16 breaks cache miss installs on windows, which blocks any PRs/commits
that change dependencies. Opened bahmutov/npm-install#146
to address the underlying issue, but in the meantime we can pin to the prior
version to get things working again.

Additional context:
- Discord: https://discord.com/channels/770287896669978684/940264701785423992/991085961972707338
- Initial PR affected: remix-run#2027
- Repro/Debug PR: remix-run#3584
@jenseng jenseng closed this Jun 28, 2022
@jenseng
Copy link
Contributor Author

jenseng commented Jun 28, 2022

@MichaelDeBoey yup, issue filed here: bahmutov/npm-install#146

Also opened this Remix PR to pin the version if it's not a quick fix: #3587

@jenseng
Copy link
Contributor Author

jenseng commented Jun 28, 2022

going to debug this a little more to see if i can figure out the right fix to be applied upstream

@jenseng jenseng reopened this Jun 28, 2022
@jenseng
Copy link
Contributor Author

jenseng commented Jun 28, 2022

closing again, i have a simpler way to repro/debug :)

@jenseng jenseng closed this Jun 28, 2022
jenseng added a commit to jenseng/remix that referenced this pull request Jun 28, 2022
v1.8.16 breaks cache miss installs on windows, which blocks any PRs/commits
that change dependencies. Opened bahmutov/npm-install#146
to address the underlying issue, but in the meantime we can pin to the prior
version to get things working again.

Additional context:
- Discord: https://discord.com/channels/770287896669978684/940264701785423992/991085961972707338
- Initial PR affected: remix-run#2027
- Repro/Debug PR: remix-run#3584
jenseng added a commit to jenseng/remix that referenced this pull request Jun 28, 2022
v1.8.16 breaks cache miss installs on windows, which blocks any PRs/commits
that change dependencies. Opened bahmutov/npm-install#146
to address the underlying issue, but in the meantime we can pin to the prior
version to get things working again.

Additional context:
- Discord: https://discord.com/channels/770287896669978684/940264701785423992/991085961972707338
- Initial PR affected: remix-run#2027
- Repro/Debug PR: remix-run#3584
jenseng added a commit to jenseng/remix that referenced this pull request Jun 28, 2022
v1.8.16 breaks cache miss installs on windows, which blocks any PRs/commits
that change dependencies. Opened bahmutov/npm-install#146
to address the underlying issue, but in the meantime we can pin to the prior
version to get things working again.

Additional context:
- Discord: https://discord.com/channels/770287896669978684/940264701785423992/991085961972707338
- Initial PR affected: remix-run#2027
- Repro/Debug PR: remix-run#3584
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.

2 participants