Skip to content

Test workflows: set correct download path for build artifacts - #763

Merged
mkernohanbc merged 1 commit into
mainfrom
bugfix/test-publish-workflows
Jun 26, 2026
Merged

Test workflows: set correct download path for build artifacts#763
mkernohanbc merged 1 commit into
mainfrom
bugfix/test-publish-workflows

Conversation

@mkernohanbc

Copy link
Copy Markdown
Contributor

Attempt to fix this job failure in the test workflows from #762: https://github.com/bcgov/design-system/actions/runs/28268823077/job/83761783124

@mkernohanbc mkernohanbc self-assigned this Jun 26, 2026
@mkernohanbc
mkernohanbc requested a review from a team as a code owner June 26, 2026 22:38
@ty2k
ty2k requested a review from Copilot June 26, 2026 22:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the two temporary “test publish” GitHub Actions workflows to download build artifacts into the directories where the subsequent npm publish steps expect the package contents to exist, addressing the job failure observed in the test workflows introduced in #762.

Changes:

  • Set actions/download-artifact path for the react components test workflow to download into packages/react-components.
  • Set actions/download-artifact path for the design tokens test workflow to download into packages/design-tokens/dist.
  • Apply the same artifact download path fix across all publish simulation jobs (next/latest, npm/GitHub Packages) in both workflows.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/test_publish_react_component_library.yaml Downloads the react-components build artifact into packages/react-components so npm publish runs against the expected working directory.
.github/workflows/test_publish_design_tokens.yaml Downloads the design-tokens build artifact into packages/design-tokens/dist so npm publish runs against the expected working directory.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ty2k ty2k left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚀

@mkernohanbc
mkernohanbc merged commit c5061e9 into main Jun 26, 2026
4 checks passed
@mkernohanbc
mkernohanbc deleted the bugfix/test-publish-workflows branch June 26, 2026 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants