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

Added missing import in pulsar_binary.py #1360

Merged
merged 2 commits into from
Jul 26, 2022

Conversation

abhisrkckl
Copy link
Contributor

@abhisrkckl abhisrkckl commented Jul 26, 2022

Fixes bug #1356

Do I need to create a test for this?

@codecov
Copy link

codecov bot commented Jul 26, 2022

Codecov Report

Merging #1360 (84f29ff) into master (bfa227e) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1360   +/-   ##
=======================================
  Coverage   61.29%   61.29%           
=======================================
  Files          90       90           
  Lines       20351    20351           
  Branches     3646     3646           
=======================================
  Hits        12475    12475           
  Misses       7089     7089           
  Partials      787      787           
Impacted Files Coverage Δ
src/pint/models/pulsar_binary.py 76.24% <100.00%> (ø)

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 bfa227e...84f29ff. Read the comment docs.

@paulray paulray merged commit 0fd7b0c into nanograv:master Jul 26, 2022
@abhisrkckl abhisrkckl deleted the fix-bug-#1356 branch July 26, 2022 21:08
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