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

Dependency cleanup; post-save test #68

Merged
merged 4 commits into from Mar 11, 2021
Merged

Dependency cleanup; post-save test #68

merged 4 commits into from Mar 11, 2021

Conversation

jayqi
Copy link
Member

@jayqi jayqi commented Mar 9, 2021

@jayqi jayqi requested review from pjbull and ejm714 March 9, 2021 06:31
@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2021

@codecov
Copy link

codecov bot commented Mar 9, 2021

Codecov Report

Merging #68 (ef62bc2) into master (3a27fa7) will not change coverage.
The diff coverage is 100.0%.

@@          Coverage Diff           @@
##           master     #68   +/-   ##
======================================
  Coverage    98.1%   98.1%           
======================================
  Files           8       8           
  Lines         382     382           
======================================
  Hits          375     375           
  Misses          7       7           
Impacted Files Coverage Δ
nbautoexport/jupyter_config.py 96.5% <100.0%> (ø)

Copy link
Contributor

@ejm714 ejm714 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 great! One optional thought on addtl checks but also good to go as is.

tests/test_post_save.py Show resolved Hide resolved
@jayqi jayqi merged commit 75212d2 into master Mar 11, 2021
@jayqi jayqi deleted the postsave-test branch March 11, 2021 06:53
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.

Clean up dependencies Write integration test that triggers post-save with Jupyter saving
2 participants