workers-sdk icon indicating copy to clipboard operation
workers-sdk copied to clipboard

🐛 BUG: Prod/release build on main seems to fail when multiple packages need releasing

Open rozenmd opened this issue 2 years ago • 3 comments

What version of Wrangler are you using?

wrangler@latest

What operating system are you using?

GitHub

Describe the Bug

https://github.com/cloudflare/wrangler2/runs/8073062908?check_suite_focus=true

rozenmd avatar Aug 29 '22 16:08 rozenmd

We should test if this is still an issue sometime.

JacobMGEvans avatar Sep 13 '22 19:09 JacobMGEvans

It's still an issue sometimes 😬

2/3 of our releases had to be re-run today

rozenmd avatar Sep 13 '22 19:09 rozenmd

@rozenmd Do you think it might have to do with the modify version script? Maybe when multiple releases happen Changeset expects monorepo behavior but that is usually through Lerna, Turborepo or the like... Instead the custom behavior we have disrupts something Changesets is doing in that time for the other packages.

      - name: Modify package.json version
        run: node .github/version-script.js

EDIT: nvm, I don't think that script would cause that.

JacobMGEvans avatar Sep 20 '22 19:09 JacobMGEvans

I was looking at this:

      - name: Create Version PR or Publish to NPM
        id: changesets
        uses: changesets/action@v1
        with:
          version: node .github/changeset-version.js
          publish: npx changeset publish

Why does node .github/changeset-version.js also do npx changeset publish? might be the issue here?

CC: @petebacondarwin

rozenmd avatar Oct 14 '22 10:10 rozenmd

Compare to miniflare, a similar monorepo, which does:

      - name: Create Version PR or Publish to NPM
        id: changesets
        uses: changesets/action@v1
        with:
          publish: npx changeset publish

rozenmd avatar Oct 14 '22 10:10 rozenmd

@rozenmd @JacobMGEvans is this still an issue?

lrapoport-cf avatar Jan 09 '23 16:01 lrapoport-cf

Not sure, I might have to set up the CI/CD locally then test it.

JacobMGEvans avatar Jan 09 '23 17:01 JacobMGEvans

Can always close and re-open when it happens again

rozenmd avatar Jan 09 '23 17:01 rozenmd

i'm good with that 👍

lrapoport-cf avatar Jan 09 '23 17:01 lrapoport-cf

reopening due to this issue occurring on release of v2.7.0.

lrapoport-cf avatar Jan 09 '23 19:01 lrapoport-cf

Closing this as it seems fixed

rozenmd avatar May 15 '23 14:05 rozenmd