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 CI workflow to push dist output to separate branch #146

Merged
merged 1 commit into from
Jul 25, 2023
Merged

Conversation

NullVoxPopuli
Copy link
Collaborator

@NullVoxPopuli NullVoxPopuli commented Jul 14, 2023

Resolves #143
Requires: #151

Because v2 addons need to be built,
we can't easily point package.json files at the git repo for easy cross-repo testing.

This push-dist workflow brings back that capability by placing the compiled assets on a "dist" branch
(configurable via the "branch" option)

@NullVoxPopuli NullVoxPopuli changed the base branch from main to add-fixture-utils July 15, 2023 19:41
@NullVoxPopuli NullVoxPopuli force-pushed the add-push-dist branch 2 times, most recently from 9daee8a to bd55162 Compare July 15, 2023 20:11
@NullVoxPopuli NullVoxPopuli marked this pull request as ready for review July 15, 2023 21:08
@NullVoxPopuli NullVoxPopuli force-pushed the add-fixture-utils branch 3 times, most recently from 387847b to 5d2aae9 Compare July 17, 2023 12:27
Base automatically changed from add-fixture-utils to main July 17, 2023 12:31
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3<% if (pnpm) {%>
- uses: NullVoxPopuli/action-setup-pnpm@v2<%} else {%>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do we need the different setup for pnpm here? Is it worth the additional complexity here?

@simonihmig simonihmig changed the title Add push dist Add CI workflow to push dist output to separate branch Jul 19, 2023
@ef4 ef4 merged commit f4f840d into main Jul 25, 2023
@ef4 ef4 deleted the add-push-dist branch July 25, 2023 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add "push-dist" workflow so folks can point to git/sha/commit for "main" development pre-publish
3 participants