ci: release workflow #106

AriPerkkio · 2024-06-28T15:32:08Z

Adds scripts for performing releases.

Start by running Prepare release PR workflow:

This workflow will bump versions and generate changelogs for all packages, except CLI. It creates a PR like chore: release @tutorialkit packages and create-tutorial, version: 0.0.1-alpha.23 AriPerkkio/tutorialkit#26
Merging the PR triggers another workflow with two jobs:
- First one creates git tags and publishes packages to npm
- It triggers second job which prepares release PR for CLI package, with the updated package versions: chore: release tutorialkit CLI, version: 0.0.1-alpha.23 AriPerkkio/tutorialkit#29

The opened PR will run CLI Integration tests (this doesn't trigger automatically on my fork though, let's re-check this on first real release 🤔)
Merging this PR will trigger another workflow that updates versions in templates, runs CLI integration tests (once again just to be sure) and finally publishes NPM package

stackblitz · 2024-06-28T15:32:11Z

Run & review this pull request in StackBlitz Codeflow.

AriPerkkio · 2024-07-01T09:14:44Z

.github/workflows/publish-release.yaml

+    permissions:
+      contents: write
+      id-token: write


These are required for --provenance flag.

AriPerkkio · 2024-07-01T09:15:24Z

.github/workflows/publish-release.yaml

+        env:
+          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}


We'll need to manually add this secret to the project.

@d3lm I think if you can add @stackblitz-gitops to the packages, then we should be able to reuse the token of @stackblitz-gitops.

@apai4 Do you think that would be fine? Or should we have a different token?

I pinged Pai about this because I cannot add NPM access tokens since I don't have permissions to generate them.

I'll create a new token for stackblitz-devops on NPM for this repo and add it to secrets.NPM_TOKEN. stackblitz-devops will need write access to the NPM package.

The token has been created and added to the repo. Let me know if it works once stackblitz-gitops is granted access

@d3lm can you grant access to stackblitz-gitops on all the packages?

Yes, I can do that!

Invite is out. I think @apai4 needs to accept it.

@d3lm do you have permissions to check if this is completed now? Publish step failed at https://github.com/stackblitz/tutorialkit/actions/runs/9791481416/job/27035364611

.github/actions/setup-and-build/action.yml

AriPerkkio · 2024-07-01T09:36:52Z

Need to recheck the integration tests again. They started to fail for me locally after rebase. Would appreciate if someone else could also try them locally. 🙌

Nemikolh

Looks really good! 💪

packages/cli/src/commands/create/index.ts

integration/cli/create-tutorial.test.ts

.github/workflows/prepare-release.yaml

.github/workflows/publish-release.yaml

.github/actions/resolve-release-version/action.yml

.github/workflows/prepare-release.yaml

.github/workflows/publish-release.yaml

d3lm

Left a few minor comments. Great work @AriPerkkio 👏

.github/actions/setup-and-build/action.yml

.github/workflows/prepare-release.yaml

.github/workflows/integration-test-cli.yaml

.github/workflows/publish-release.yaml

.gitignore

integration/cli/create-tutorial.test.ts

packages/cli/scripts/update-template.js

packages/cli/src/commands/create/index.ts

packages/cli/src/commands/create/dependencies.ts

packages/cli/src/commands/create/package-manager.ts

packages/cli/package.json

AriPerkkio · 2024-07-02T15:00:00Z

Verified that everything still works after PR changes in my fork ✅

packages/cli/scripts/update-template.js

Nemikolh

Fantastic work on this @AriPerkkio ! 😃 👏

d3lm

Awesome work! Only minor remarks.

.github/workflows/prepare-release.yaml

.github/workflows/publish-release.yaml

integration/cli/__snapshots__/npm-created.json

integration/cli/__snapshots__/pnpm-created.json

integration/cli/__snapshots__/yarn-created.json

d3lm

Great work 💪 👏 🔥

AriPerkkio force-pushed the ci/publish-scripts branch 2 times, most recently from dd152ad to 29da35d Compare July 1, 2024 08:58

SamVerschueren and others added 2 commits July 1, 2024 12:00

ci: add automated releases

66ef9e0

ci: reusable setup action

cddfde9

AriPerkkio force-pushed the ci/publish-scripts branch 2 times, most recently from 24b3bf5 to df1bca6 Compare July 1, 2024 09:13

AriPerkkio commented Jul 1, 2024

View reviewed changes

ci: release workflow

4317768

AriPerkkio force-pushed the ci/publish-scripts branch from df1bca6 to 4317768 Compare July 1, 2024 09:20

AriPerkkio marked this pull request as ready for review July 1, 2024 09:20

AriPerkkio requested review from SamVerschueren, Nemikolh and d3lm July 1, 2024 09:20

AriPerkkio commented Jul 1, 2024

View reviewed changes

.github/actions/setup-and-build/action.yml Show resolved Hide resolved

Nemikolh reviewed Jul 1, 2024

View reviewed changes

fix: code review

bf924d7

d3lm reviewed Jul 1, 2024

View reviewed changes

fix: code review

626b996

AriPerkkio commented Jul 2, 2024

View reviewed changes

packages/cli/src/commands/create/dependencies.ts Outdated Show resolved Hide resolved

fix: code review

64c43e2

AriPerkkio force-pushed the ci/publish-scripts branch from 54a386c to 64c43e2 Compare July 2, 2024 09:09

AriPerkkio added 2 commits July 2, 2024 12:48

fix: code review

6de490e

fix: code review

eba02b2

AriPerkkio force-pushed the ci/publish-scripts branch from 32fc233 to eba02b2 Compare July 2, 2024 14:21

Nemikolh reviewed Jul 2, 2024

View reviewed changes

packages/cli/src/commands/create/package-manager.ts Outdated Show resolved Hide resolved

Nemikolh reviewed Jul 2, 2024

View reviewed changes

packages/cli/package.json Show resolved Hide resolved

fix: code review

32e32ec

fix: code review

908098c

Nemikolh reviewed Jul 2, 2024

View reviewed changes

packages/cli/scripts/update-template.js Outdated Show resolved Hide resolved

fix: code review

1a3356c

Nemikolh approved these changes Jul 3, 2024

View reviewed changes

Merge branch 'main' into ci/publish-scripts

d9fd843

d3lm reviewed Jul 3, 2024

View reviewed changes

fix: code review

7bd4114

Nemikolh requested a review from d3lm July 3, 2024 09:39

d3lm approved these changes Jul 3, 2024

View reviewed changes

Nemikolh merged commit 559b8e4 into stackblitz:main Jul 3, 2024
8 checks passed

AriPerkkio deleted the ci/publish-scripts branch July 3, 2024 09:43

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ci: release workflow #106

ci: release workflow #106

AriPerkkio commented Jun 28, 2024 •

edited

Loading

stackblitz bot commented Jun 28, 2024

AriPerkkio Jul 1, 2024

AriPerkkio Jul 1, 2024

Nemikolh Jul 1, 2024

d3lm Jul 1, 2024

apai4 Jul 1, 2024

apai4 Jul 1, 2024

Nemikolh Jul 1, 2024

d3lm Jul 2, 2024

d3lm Jul 3, 2024

AriPerkkio Jul 4, 2024

AriPerkkio commented Jul 1, 2024 •

edited

Loading

Nemikolh left a comment

d3lm left a comment

AriPerkkio commented Jul 2, 2024

Nemikolh left a comment

d3lm left a comment

d3lm left a comment

ci: release workflow #106

ci: release workflow #106

Conversation

AriPerkkio commented Jun 28, 2024 • edited Loading

stackblitz bot commented Jun 28, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

AriPerkkio commented Jul 1, 2024 • edited Loading

Nemikolh left a comment

Choose a reason for hiding this comment

d3lm left a comment

Choose a reason for hiding this comment

AriPerkkio commented Jul 2, 2024

Nemikolh left a comment

Choose a reason for hiding this comment

d3lm left a comment

Choose a reason for hiding this comment

d3lm left a comment

Choose a reason for hiding this comment

AriPerkkio commented Jun 28, 2024 •

edited

Loading

AriPerkkio commented Jul 1, 2024 •

edited

Loading