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

ci: switch to shared actions #357

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

nathanosdev
Copy link
Member

@nathanosdev nathanosdev commented Aug 28, 2024

Merging this will require coordination. Since the names of the jobs have changed, we will need IDX to update the names of the required jobs so that the CI can pass. Once that change is made by IDX, no other branch will be able to pass the CI pipeline until this one is merged.

@nathanosdev nathanosdev self-assigned this Aug 28, 2024
@nathanosdev nathanosdev force-pushed the nathan/switch-shared-pnpm-action branch from f1c24ae to 027f977 Compare November 25, 2024 12:21
@nathanosdev nathanosdev changed the title ci: switch to shared pnpm action ci: switch to shared actions Nov 25, 2024
@nathanosdev nathanosdev force-pushed the nathan/switch-shared-pnpm-action branch 2 times, most recently from 90613a9 to a6a5010 Compare November 25, 2024 16:17
@nathanosdev nathanosdev reopened this Nov 25, 2024
@nathanosdev nathanosdev force-pushed the nathan/switch-shared-pnpm-action branch from a6a5010 to 715ddeb Compare November 26, 2024 11:40
@nathanosdev nathanosdev force-pushed the nathan/switch-shared-pnpm-action branch 5 times, most recently from 8709015 to befac51 Compare December 9, 2024 12:05
@nathanosdev nathanosdev force-pushed the nathan/switch-shared-pnpm-action branch from befac51 to 0d56e53 Compare December 10, 2024 13:26
@nathanosdev nathanosdev marked this pull request as ready for review December 10, 2024 13:34
@nathanosdev nathanosdev requested a review from a team as a code owner December 10, 2024 13:34
Copy link
Contributor

@przydatek przydatek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@@ -1,42 +1,14 @@
name: Update changelog
name: generate_changelog
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: should this file be renamed too? (for consistency)

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.

2 participants