Skip to content

Conversation

@jenshnielsen
Copy link
Collaborator

@jenshnielsen jenshnielsen commented Dec 6, 2022

This is useful for kicking of postprocessing, backup, upload etc.

  • Document
  • Test using a entry point that logs the export event

@codecov
Copy link

codecov bot commented Dec 6, 2022

Codecov Report

Merging #4857 (5526303) into master (c220bf7) will increase coverage by 0.01%.
The diff coverage is 88.00%.

@@            Coverage Diff             @@
##           master    #4857      +/-   ##
==========================================
+ Coverage   68.30%   68.31%   +0.01%     
==========================================
  Files         339      340       +1     
  Lines       32128    32150      +22     
==========================================
+ Hits        21944    21964      +20     
- Misses      10184    10186       +2     

@jenshnielsen jenshnielsen force-pushed the add_export_entry_point branch from 3e37875 to 1593a26 Compare December 13, 2022 12:30
@jenshnielsen jenshnielsen marked this pull request as ready for review December 13, 2022 14:17
Copy link
Contributor

@Dominik-Vogel Dominik-Vogel left a comment

Choose a reason for hiding this comment

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

This is beautiful!
I have tested it and it works ✔️

@jenshnielsen jenshnielsen force-pushed the add_export_entry_point branch from 48dd604 to 5526303 Compare December 20, 2022 12:20
@jenshnielsen jenshnielsen merged commit 3f78c23 into microsoft:master Dec 20, 2022
@jenshnielsen jenshnielsen deleted the add_export_entry_point branch December 20, 2022 12:52
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.

2 participants