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

ci(chromatic): use diffThreshold value saved in secrets #5806

Merged
merged 2 commits into from
Nov 24, 2022

Conversation

benelan
Copy link
Member

@benelan benelan commented Nov 23, 2022

Related Issue: #

Summary

I forgot to pass the diff threshold GH secret to the chromatic workflow's environment variables 😅
That explains why bumping up the value wasn't doing anything

@benelan benelan requested a review from a team as a code owner November 23, 2022 22:51
@github-actions github-actions bot added the chore Issues with changes that don't modify src or test files. label Nov 23, 2022
Copy link
Member

@driskull driskull left a comment

Choose a reason for hiding this comment

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

👍

@benelan benelan merged commit d37cd8d into master Nov 24, 2022
@benelan benelan deleted the benelan/actually-use-diff-threshold-secret branch November 24, 2022 01:16
benelan added a commit that referenced this pull request Nov 28, 2022
* master:
  docs: update component READMEs (#5810)
  ci(chromatic): use diffThreshold value saved in secrets (#5806)
  1.0.0-next.642
  fix(combobox): 5540 - handle focus (#5774)
  1.0.0-next.641
  fix(tree-item): Allow space and enter key events when selectionMode is "none" (#5800)
  Update CONTRIBUTING.md
  1.0.0-next.640
  fix(input-date-picker): display updated valueAsDate in the two range inputs (#5758)
  docs(tree): Update interface for selectionMode. (#5801)
  docs: update component READMEs (#5804)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Issues with changes that don't modify src or test files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants