Skip to content
This repository was archived by the owner on Jul 12, 2024. It is now read-only.

Conversation

@drdavella
Copy link
Contributor

@drdavella drdavella commented Nov 3, 2017

It doesn't matter much for astropy, but there's a bug when reading change logs that only have a single milestone in them (e.g. for a new repository). I added some unit tests to increase coverage and expose the bug (and I fixed it, too).

@coveralls
Copy link

Coverage Status

Coverage increased (+5.7%) to 74.005% when pulling cd1b34e on drdavella:add-changelog-tests into e46245d on astropy:master.

@drdavella
Copy link
Contributor Author

Also, the comments in the fake change logs are just for fun. They are not intended as commentary about anyone here, or about this or any related projects 😅 .

Copy link
Member

@astrofrog astrofrog left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@pllim pllim merged commit 0e957d3 into astropy:master Nov 3, 2017
@drdavella drdavella deleted the add-changelog-tests branch November 3, 2017 19:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants