Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

ci: chromatic workflow #23

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

ci: chromatic workflow #23

wants to merge 9 commits into from

Conversation

SergeyRoyt
Copy link
Collaborator

@SergeyRoyt SergeyRoyt commented Jan 2, 2024 •
edited
Loading

Basically we have two options here:

  • Combining all of these into on workflow
    • Main advantage - looks better :)
  • Keeping it in 3 (currently) separate workflows
    • Main advantage - runs concurrently

@SergeyRoyt SergeyRoyt self-assigned this Jan 2, 2024
SergeyRoyt
SergeyRoyt commented Jan 4, 2024
View reviewed changes
.github/workflows/chromatic.yml
run: |
yarn install --frozen-lockfile
yarn build
- name: Publish to Chromatic
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

env: |
secrets: ${{ github.secrets }}
...
smth like this
instead of exposion of chromatic tokens

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@SergeyRoyt SergeyRoyt

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@SergeyRoyt