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

The npm scripts "publish:hex" defined in sync-service and elixir-client publish on every commit to main #2205

Open
alco opened this issue Dec 20, 2024 · 0 comments

Comments

@alco
Copy link
Member

alco commented Dec 20, 2024

When a new commit is merged to main and there are no any changesets in .changeset/ yet, the publish:hex scripts will run and try to publish to Hex using the same package version but built from a different commit.

This isn't a critical problem because publish an existing package can only succeed when the --replace flag is provided. However, we're wasting compute and prolonging the CI workflow's runtime needlessly with this setup.

https://github.com/electric-sql/electric/actions/runs/12420871198/job/34679331020
image

Instead we need an explicit CI workflow that will trigger new version publishing to hex.pm when it detects a new Git tag on the repo, similarly to how we need that for release builds on Buildkite - #2204.

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

No branches or pull requests

1 participant