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

Fix for issue #506 #542

Closed
wants to merge 5 commits into from
Closed

Conversation

jaredbriskman
Copy link

This changes the spelling of millennium in the default definition file, raised by issue #506 . It is a breaking change for anything that was defined with the misspelled version, as these versions of the unit were removed. To maintain backwards compatibility, these could be added back as aliases. It also has not been changed in the default_en_0.6 definition file.

@jaredbriskman jaredbriskman reopened this Aug 14, 2017
@hgrecco
Copy link
Owner

hgrecco commented Aug 19, 2017

bors r+

bors bot added a commit that referenced this pull request Aug 19, 2017
540: Update list of similar units implementations r=hgrecco

I added astropy.units and yt, as those were missing from the list.

Speaking of which, has there been any talk of maybe combining some of these packages? It feels a bit like the number of these is growing exponentially.

542: Fix for issue #506 r=hgrecco

This changes the spelling of **millennium** in the default definition file, raised by issue #506 . It is a breaking change for anything that was defined with the misspelled version, as these versions of the unit were removed. To maintain backwards compatibility, these could be added back as aliases. It also has not been changed in the default_en_0.6 definition file.
@bors
Copy link
Contributor

bors bot commented Aug 19, 2017

Build failed (retrying...)

bors bot added a commit that referenced this pull request Aug 19, 2017
542: Fix for issue #506 r=hgrecco

This changes the spelling of **millennium** in the default definition file, raised by issue #506 . It is a breaking change for anything that was defined with the misspelled version, as these versions of the unit were removed. To maintain backwards compatibility, these could be added back as aliases. It also has not been changed in the default_en_0.6 definition file.
@bors
Copy link
Contributor

bors bot commented Aug 19, 2017

Build failed

@crusaderky
Copy link
Contributor

Fixed by #811 a few months ago; please close

@hgrecco hgrecco closed this Dec 13, 2019
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.

3 participants