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

[DRAFT] chore: set up auto for versioning/release management #497

Closed
wants to merge 6 commits into from

Conversation

hydrosquall
Copy link
Member

@hydrosquall hydrosquall commented May 7, 2022

Motivation

Request from @domoritz as part of the Vega org-wide automation effort. Note this is the first time we're applying auto to a Monorepo (with multiple yarn workspaces) in 1 repo in the Vega org, but it's possible (see this repo for reference).

See vega/vega-lite#7984 for a similar PR.

Changes

Pre-merge checklist for a repo admin

📦 Published PR as canary version: 7.5.2--canary.497.1504252.0

✨ Test out this PR locally via:

npm install react-vega@7.5.2--canary.497.1504252.0
# or 
yarn add react-vega@7.5.2--canary.497.1504252.0

@hydrosquall hydrosquall self-assigned this May 7, 2022
@hydrosquall hydrosquall force-pushed the cameron.yick/feature/add-auto branch 3 times, most recently from c7b661f to e3127d1 Compare May 7, 2022 21:45
@domoritz domoritz requested a review from kristw May 7, 2022 23:47
@domoritz
Copy link
Member

domoritz commented May 7, 2022

@kristw is maintaining this repo so I will let him decide how we wants to handle things.

@hydrosquall hydrosquall force-pushed the cameron.yick/feature/add-auto branch from 74cd718 to 4fdc694 Compare May 9, 2022 14:37
…version in actions

build: enable lerna automation tokens to run with correct permissions

build: use explicit node version

build: swtich publishConfig from .org to .com

build: include NPM token in addition to auth token

build: include NPM token in addition to auth token

build: use npm set to modify lerna publish credentials
@hydrosquall hydrosquall force-pushed the cameron.yick/feature/add-auto branch from 4fdc694 to 1504252 Compare May 10, 2022 00:02
@hydrosquall hydrosquall added skip-release do not publish new NPM builds for this package released this PR has been released to NPM and removed skip-release do not publish new NPM builds for this package released this PR has been released to NPM labels May 10, 2022
@kristw
Copy link
Member

kristw commented Jun 29, 2022

Appreciate the effort in adding automation. Let me take some time to read through.

@hydrosquall
Copy link
Member Author

Given the complications that arose with Auto in the future ( see vega/vega-lite#8370 ), I think it is best if we close this PR for now. If we revisit automatic releases in the future, we may be better off using release-it, which is more actively maintained and does not have this version drift bug (no action needed, just FYI @domoritz ).

@hydrosquall hydrosquall deleted the cameron.yick/feature/add-auto branch February 27, 2023 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants