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

Add automated legacy releases to *.x branches #224

Merged
merged 3 commits into from
Jun 9, 2023

Conversation

jpwilliams
Copy link
Member

Summary

Use *.x branches such as 1.x with changesets to create automated release PRs, similar to those to @latest.

Here, we add a small script to generate a dynamic config for these branches so that changesets knows what to compare against.

Also includes a small change to rename release PRs to either Release @latest or (for example) Release @1.x.

Related

@jpwilliams jpwilliams self-assigned this Jun 9, 2023
@changeset-bot
Copy link

changeset-bot bot commented Jun 9, 2023

⚠️ No Changeset found

Latest commit: 806d07d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

jpwilliams added a commit that referenced this pull request Jun 9, 2023
## Summary

This handy action allows us to backport a PR to a particular branch when
merged using labels.


![image](https://github.com/inngest/inngest-js/assets/1736957/54f4295d-8bc8-4341-a277-0344f9e86615)

Just add the label, then when the PR is merged a new PR will be opened
to the relevant branch (e.g. `1.x`) to backport the squashed commit.

This does not manage releasing; we'll use changesets for that.

## Related

- #224
inngest-release-bot pushed a commit that referenced this pull request Jun 9, 2023
## Summary

This handy action allows us to backport a PR to a particular branch when
merged using labels.

![image](https://github.com/inngest/inngest-js/assets/1736957/54f4295d-8bc8-4341-a277-0344f9e86615)

Just add the label, then when the PR is merged a new PR will be opened
to the relevant branch (e.g. `1.x`) to backport the squashed commit.

This does not manage releasing; we'll use changesets for that.

## Related

- #224

(cherry picked from commit efc37dc)
@jpwilliams jpwilliams added the backport v1.x Backport this PR to v1.x when merged. label Jun 9, 2023
@jpwilliams jpwilliams merged commit 634e2d6 into main Jun 9, 2023
19 checks passed
@jpwilliams jpwilliams deleted the chore/add-automated-legacy-releases branch June 9, 2023 19:55
inngest-release-bot pushed a commit that referenced this pull request Jun 9, 2023
## Summary

Use `*.x` branches such as `1.x` with changesets to create automated
release PRs, similar to those to `@latest`.

Here, we add a small script to generate a dynamic config for these
branches so that changesets knows what to compare against.

Also includes a small change to rename release PRs to either `Release
@latest` or (for example) `Release @1.x`.

## Related

- #223

(cherry picked from commit 634e2d6)
jpwilliams added a commit that referenced this pull request Jun 9, 2023
Backport 634e2d6 from #224.

Co-authored-by: Jack Williams <jack@inngest.com>
jpwilliams added a commit that referenced this pull request Jun 12, 2023
…gs (#236)

## Summary

The release script in #224 and #230 almost worked.

We see in run to release a `v1.x` backport that the following command
failed:

https://github.com/inngest/inngest-js/actions/runs/5244055017/jobs/9469531269
```
$ npm dist-tag add inngest@2.0.1 latest

npm ERR! code E401
npm ERR! 401 Unauthorized - PUT https://registry.yarnpkg.com/-/package/inngest/dist-tags/latest
```

Looks like it's defaulting to using Yarn's registry URL instsead of
npm's. We get around this during publishing as we specify
`publishConfig.registry` in our `package.json`, but it seems that `npm
dist-tag` doesn't obey this same property.

Let's manually set the registry for this particular command to ensure it
tries to affect the right place.

## Related

- #224 
- #230
inngest-release-bot pushed a commit that referenced this pull request Jun 12, 2023
…gs (#236)

## Summary

The release script in #224 and #230 almost worked.

We see in run to release a `v1.x` backport that the following command
failed:

https://github.com/inngest/inngest-js/actions/runs/5244055017/jobs/9469531269
```
$ npm dist-tag add inngest@2.0.1 latest

npm ERR! code E401
npm ERR! 401 Unauthorized - PUT https://registry.yarnpkg.com/-/package/inngest/dist-tags/latest
```

Looks like it's defaulting to using Yarn's registry URL instsead of
npm's. We get around this during publishing as we specify
`publishConfig.registry` in our `package.json`, but it seems that `npm
dist-tag` doesn't obey this same property.

Let's manually set the registry for this particular command to ensure it
tries to affect the right place.

## Related

- #224
- #230

(cherry picked from commit f33464f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport v1.x Backport this PR to v1.x when merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants