Skip to content

Update constant.py to match repr to attributes. - #4146

Closed
DanielLenz wants to merge 1 commit into
astropy:masterfrom
DanielLenz:patch-1
Closed

DanielLenz wants to merge 1 commit into
astropy:masterfrom
DanielLenz:patch-1

Conversation

@DanielLenz

Copy link
Copy Markdown

Rename Error -> Uncertainty and Units -> Unit in the repr to match the attributes.
Requested in #4145

Rename Error -> Uncertainty and Units -> Unit to match the attributes.
@DanielLenz DanielLenz changed the title Update constant.py Update constant.py to match repr to attributes. Sep 11, 2015
@mhvk

mhvk commented Sep 11, 2015

Copy link
Copy Markdown
Contributor

+1. Note the build failures due to omitting to update the documentation docs/constants/index.html. I'll leave it to @embray on whether this needs a changelog entry.

@DanielLenz DanielLenz closed this Sep 11, 2015
@DanielLenz
DanielLenz deleted the patch-1 branch September 11, 2015 15:50
@pllim

pllim commented Sep 11, 2015

Copy link
Copy Markdown
Member

It looks like the branch was deleted without merging?

@DanielLenz

Copy link
Copy Markdown
Author

I messed up the branches, so I created a new pull request, including the changes in the docs. First contribution, please bear with me.

@pllim

pllim commented Sep 11, 2015

Copy link
Copy Markdown
Member

@dlenz-bonn , no problem. Can you please link to your new PR as a comment in #4145, so there is a clear cross-reference for reviewers? Thank you for contributing!

@pllim

pllim commented Sep 11, 2015

Copy link
Copy Markdown
Member

Ah, never mind. I see the new PR now.

@embray

embray commented Sep 14, 2015

Copy link
Copy Markdown
Member

Next time just let me know--in the vast majority of cases the git issues can be sorted out without creating a new PR.

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.

4 participants