Skip to content

chore: speed up chromatic ci - #904

Merged
greyscaled merged 1 commit into
mainfrom
vapurrmaid/chromatic-speed
Apr 6, 2022
Merged

greyscaled merged 1 commit into
mainfrom
vapurrmaid/chromatic-speed

Conversation

@greyscaled

Copy link
Copy Markdown
Contributor

This is an oversight from #896 . It turns out that because we use the GitHub integration with Chromatic, we don't need to wait for the results to be reported in the action - they get reported in the other checks created by Chromatic.

This option was spit-out in a check:

https://github.com/coder/coder/runs/5859427236?check_suite_focus=true#step:4:38

Relates to #444

This is an oversight from #896 . It turns out that because we use the GitHub integration with Chromatic, we don't need to wait for the results to be reported in the action - they get reported in the other checks created by Chromatic.

This option was spit-out in a check:

https://github.com/coder/coder/runs/5859427236?check_suite_focus=true#step:4:38

Relates to #444
@greyscaled
greyscaled requested a review from deansheather April 6, 2022 22:34
@greyscaled greyscaled self-assigned this Apr 6, 2022
@greyscaled
greyscaled enabled auto-merge (squash) April 6, 2022 22:35
@codecov

codecov Bot commented Apr 6, 2022 •

Copy link
Copy Markdown

Codecov Report

Merging #904 (4032205) into main (6d40f34) will decrease coverage by 0.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #904      +/-   ##
==========================================
- Coverage   66.19%   66.00%   -0.20%     
==========================================
  Files         126      126              
  Lines       12754    12754              
==========================================
- Hits         8443     8418      -25     
- Misses       3413     3429      +16     
- Partials      898      907       +9     
Flag Coverage Δ
unittest-go- 65.26% <ø> (-0.22%) ⬇️
unittest-go-macos-latest 52.73% <ø> (+0.04%) ⬆️
unittest-go-ubuntu-latest 55.18% <ø> (-0.18%) ⬇️
unittest-go-windows-2022 52.07% <ø> (-0.03%) ⬇️
Impacted Files Coverage Δ
provisioner/echo/serve.go 56.80% <0.00%> (-2.41%) ⬇️
coderd/workspaceresources.go 59.34% <0.00%> (-1.63%) ⬇️
peer/conn.go 77.15% <0.00%> (-1.53%) ⬇️
coderd/provisionerdaemons.go 58.51% <0.00%> (-1.52%) ⬇️
provisionerd/provisionerd.go 79.44% <0.00%> (-0.89%) ⬇️
cli/cliui/provisionerjob.go 78.57% <0.00%> (+2.14%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6d40f34...4032205. Read the comment docs.

@greyscaled
greyscaled merged commit 579fd4b into main Apr 6, 2022
@greyscaled
greyscaled deleted the vapurrmaid/chromatic-speed branch April 6, 2022 22:40
@misskniss misskniss added this to the V2 Beta milestone May 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants