You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@code-asher pointed out a bug with our npm publish. Based on the way download_artifact works in our lib.sh script, it always grabs the binary based on the latest release branch (i.e. v4.0.1). This means we've been publishing PR builds and beta builds using the same binary as latest 🤦♂️
Solution
We're going to fix this by moving the workflows into one job and moving the switch logic into the bash script. We arrived at this decision because it's the best solution given the way download_artifact works and how we utilize GitHub artifacts.
TODOs
combine npm workflows into one job in ci.yaml
add logic to bash script publish-npm to check which values to use based on environment variables in job
modify download_artifact to take an argument
make diagram that shows each of the different scenarios ("staging", "dev" and "production")
@code-asher pointed out a bug with our npm publish. Based on the way
download_artifact
works in ourlib.sh
script, it always grabs the binary based on the latest release branch (i.e.v4.0.1
). This means we've been publishingPR builds
andbeta
builds using the same binary aslatest
🤦♂️Solution
We're going to fix this by moving the workflows into one job and moving the switch logic into the bash script. We arrived at this decision because it's the best solution given the way
download_artifact
works and how we utilize GitHub artifacts.TODOs
ci.yaml
publish-npm
to check which values to use based on environment variables in jobdownload_artifact
to take an argumentResources
Notes
ENVIRONMENT: ${{ github.event_name == 'push' && github.ref == 'refs/heads/master' && "production" || "staging" }}
if: ${{ github.event_name == 'push' && github.ref == 'refs/heads/master' }}
GITHUB_EVENT_NAME
GITHUB_REF
Read here for default envs: https://docs.github.com/en/actions/learn-github-actions/environment-variables#default-environment-variables
The text was updated successfully, but these errors were encountered: