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

chore(canary): Avoid workspace:* in published package.json files #10532

Merged
merged 1 commit into from
May 1, 2024

Conversation

Josh-Walker-GM
Copy link
Collaborator

Problem
It appears leaving these workspace:* values in for peer dependencies can result in:

➤ YN0001: │ TypeError: Cannot read properties of null (reading 'set')

This was happening to me today. We can simply replace these with the canary version we are publishing.

We already avoid workspace:* in our stable releases.

Changes
Updates the existing shell script to also replace all occurrences of workspace:* with the canary version.

@Josh-Walker-GM Josh-Walker-GM added release:chore This PR is a chore (means nothing for users) changesets-ok Override the changesets check labels May 1, 2024
@Josh-Walker-GM Josh-Walker-GM added this to the chore milestone May 1, 2024
@Josh-Walker-GM Josh-Walker-GM self-assigned this May 1, 2024
@Josh-Walker-GM Josh-Walker-GM merged commit 0288d53 into main May 1, 2024
46 checks passed
@Josh-Walker-GM Josh-Walker-GM deleted the jgmw/canary-publishing-workspace-replacement branch May 1, 2024 22:35
@Josh-Walker-GM
Copy link
Collaborator Author

Merging this right in was a reasonable way to test it actually worked. It appears to have done so - the published package.json files list the canary version as the workspace:* dependency and my issue upgrading to canary is no longer occurring.

dac09 added a commit that referenced this pull request May 2, 2024
…auth-reauthenticate-refactor

* 'main' of github.com:redwoodjs/redwood:
  chore(canary): Avoid `workspace:*` in published package.json files (#10532)
Josh-Walker-GM added a commit that referenced this pull request May 2, 2024
…10532)

**Problem**
It appears leaving these `workspace:*` values in for peer dependencies
can result in:
```
➤ YN0001: │ TypeError: Cannot read properties of null (reading 'set')
```
This was happening to me today. We can simply replace these with the
canary version we are publishing.

We already avoid `workspace:*` in our stable releases.

**Changes**
Updates the existing shell script to also replace all occurrences of
`workspace:*` with the canary version.
dac09 added a commit to dac09/redwood that referenced this pull request May 2, 2024
…e-middleware-client

* 'main' of github.com:redwoodjs/redwood:
  chore(canary): Avoid `workspace:*` in published package.json files (redwoodjs#10532)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changesets-ok Override the changesets check release:chore This PR is a chore (means nothing for users)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant