-
Notifications
You must be signed in to change notification settings - Fork 34
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
Testing Astropy6rc [do not merge] #390
Conversation
Edited the related decorators to use them.
* Mark additional doctests with required packages: synphot, scipy * Rename utils.requires and utils.optional to utils.required_packages and utils.optional_packages * Rename utils.decorators.optional to utils.decorators.optionally_uses * Prefer use of unittest.mock.patch to test optional behavior when a package is available to the testing environment
…py into optional-dependencies-2023.07
…py into optional-dependencies-2023.07
Hello @mkelley! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
|
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #390 +/- ##
===========================================
+ Coverage 76.84% 86.89% +10.05%
===========================================
Files 78 81 +3
Lines 6974 7096 +122
===========================================
+ Hits 5359 6166 +807
+ Misses 1615 930 -685 ☔ View full report in Codecov by Sentry. |
No description provided.