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

Remove unnecessary imports from tests #1413

Merged
merged 26 commits into from
Oct 5, 2022

Conversation

abhisrkckl
Copy link
Contributor

I am only removing unnecessary imports from tests. No other change is made.

…servatory.py test_observatory_metadata.py test_observatory_envar.py
…sions.py test_model_wave.py test_model_manual.py test_model_derivatives.py
…st_early_chime_data.py test_downhill_fitter.py test_dmxrange_add_sub.py
…ties.py test_density.py test_ddk.py test_dd.py
@codecov
Copy link

codecov bot commented Oct 5, 2022

Codecov Report

Base: 62.74% // Head: 62.74% // No change to project coverage 👍

Coverage data is based on head (2698a61) compared to base (16d78a3).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1413   +/-   ##
=======================================
  Coverage   62.74%   62.74%           
=======================================
  Files          89       89           
  Lines       20420    20420           
  Branches     3505     3505           
=======================================
  Hits        12812    12812           
  Misses       6816     6816           
  Partials      792      792           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@abhisrkckl abhisrkckl added the awaiting review This PR needs someone to review it so it can be merged label Oct 5, 2022
@paulray paulray merged commit 683da6a into nanograv:master Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review This PR needs someone to review it so it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants