Skip to content

Conversation

greyscaled
Copy link
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
Copy link

codecov bot commented Apr 6, 2022

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