Skip to content

Conversation

@AlexNDRmac
Copy link
Contributor

  • Added action to store images tests/Images/ActualOutput/* as build artifacts

@AlexNDRmac
Copy link
Contributor Author

@antonfirsov - PR related to this discussion
If some job fail - this step store all images as build artifact to an action result.

@codecov
Copy link

codecov bot commented Dec 2, 2020

Codecov Report

Merging #1452 (2ffc4e8) into master (b27581a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1452   +/-   ##
=======================================
  Coverage   83.66%   83.66%           
=======================================
  Files         736      736           
  Lines       32012    32012           
  Branches     3609     3609           
=======================================
  Hits        26782    26782           
  Misses       4516     4516           
  Partials      714      714           
Flag Coverage Δ
unittests 83.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


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 b27581a...2ffc4e8. Read the comment docs.

@JimBobSquarePants
Copy link
Member

@AlexNDRmac That's an awesome addition. Thanks!

@JimBobSquarePants JimBobSquarePants merged commit 14c541c into SixLabors:master Dec 2, 2020
@AlexNDRmac AlexNDRmac deleted the ci-save-images-for-failed-jobs branch December 2, 2020 12:36
@antonfirsov
Copy link
Member

@AlexNDRmac thanks for pushing all these changes through!

@AlexNDRmac
Copy link
Contributor Author

Feel free to ping me if you would have any issues with mac/linux :)

JimBobSquarePants added a commit that referenced this pull request Mar 13, 2021
Add action to save Output Images for failed tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants