celo-org / celo-monorepo

Official repository for core projects comprising the Celo platform
https://celo.org
Apache License 2.0
684 stars 360 forks source link

Update npm package versioning script to be compatible with other packages #11041

Closed soloseng closed 2 weeks ago

soloseng commented 3 weeks ago

Description

Currently, the protocol-devchain-anvil CI script publishes the devchain-anvil package with a version equal to the PR number. This can cause issues, as PRs are not always merged linearly. (e.g.: when PR#11034 was merged after PR#11035).

The proposed changes will use the determine-release-version script to determine the version.

soloseng commented 2 weeks ago

Did you test this locally, or more generally, is there a way to test this before it's merged?

I just added a workflow dispatch to run this WF manually. It can be used to test pre-merge releases.