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

Fix React components library npm publish workflow logic #425

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

ty2k
Copy link
Contributor

@ty2k ty2k commented Jul 24, 2024

This PR combines the if statements in the new workflow from #424 because the original caused a broken Action run here: https://github.com/bcgov/design-system/actions/runs/10085118710

GitHub Action run failed workflow screenshot

The formatting changes in the rest of the file are automatic from VS Code, which I trust better than my own eye to format YAML like this. 🥴

@ty2k ty2k added the bug Something isn't working label Jul 24, 2024
@ty2k ty2k added this to the Components v0.2.0 milestone Jul 24, 2024
@ty2k ty2k requested a review from mkernohanbc July 24, 2024 22:58
@ty2k ty2k self-assigned this Jul 24, 2024
@ty2k ty2k merged commit 9247381 into main Jul 24, 2024
2 checks passed
@ty2k ty2k deleted the bugfix/combine-if-statements-in-publish-npm-workflow branch July 24, 2024 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants