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

Mag reynolds #820

Merged
merged 62 commits into from
Jun 2, 2020
Merged

Mag reynolds #820

merged 62 commits into from
Jun 2, 2020

Conversation

KhalilBryant
Copy link
Contributor

@KhalilBryant KhalilBryant commented May 29, 2020

KhalilBryant and others added 30 commits May 21, 2020 14:52
@pep8speaks
Copy link

pep8speaks commented May 29, 2020

Hello @KhalilBryant! Thanks for updating your pull request.

Congratulations! There are no PEP8 issues in this pull request. 😸

Comment last updated at 2020-05-31 18:02:47 UTC

@codecov
Copy link

codecov bot commented May 29, 2020

Codecov Report

Merging #820 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #820   +/-   ##
=======================================
  Coverage   95.87%   95.88%           
=======================================
  Files          56       56           
  Lines        5117     5124    +7     
=======================================
+ Hits         4906     4913    +7     
  Misses        211      211           
Impacted Files Coverage Δ
plasmapy/formulary/dimensionless.py 100.00% <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 11ecb88...0a39665. Read the comment docs.

@StanczakDominik
Copy link
Member

I merged the PR for Reynolds, could you merge those changes in? I squashed them into a single commit so as to make the git history tree less verbose.

Copy link
Member

@StanczakDominik StanczakDominik left a comment

Choose a reason for hiding this comment

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

LGTM!

@StanczakDominik StanczakDominik merged commit 25c8127 into PlasmaPy:master Jun 2, 2020
@KhalilBryant KhalilBryant deleted the MagReynolds branch June 2, 2020 17:50
@rocco8773 rocco8773 added the plasmapy.formulary Related to the plasmapy.formulary subpackage label Jul 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plasmapy.formulary Related to the plasmapy.formulary subpackage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants