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

Adding #51 to changelog #54

Merged
merged 2 commits into from
Jul 28, 2016
Merged

Adding #51 to changelog #54

merged 2 commits into from
Jul 28, 2016

Conversation

esc
Copy link
Member

@esc esc commented Jul 28, 2016

Was missing.

@esc
Copy link
Member Author

esc commented Jul 28, 2016

Can someone please merge this!

@@ -1205,6 +1205,8 @@ Changelog

* v0.10.1 - Not released yet

* Unpinn python-blosc (#51)
Copy link
Member

Choose a reason for hiding this comment

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

Unpinn is a verb?

Copy link
Contributor

Choose a reason for hiding this comment

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

Unpin can be used as a verb if you want, but I assume that you would want to use past tense in release notes/change log: unpinned.

Copy link
Member Author

Choose a reason for hiding this comment

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

It's English, man! Just make it up as you go along!! :)

Copy link
Member Author

Choose a reason for hiding this comment

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

The changelog uses present-tense until now, let's stay consistent?

Copy link
Contributor

Choose a reason for hiding this comment

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

Oops, I should have picked up on that!

@esc
Copy link
Member Author

esc commented Jul 28, 2016

I fixored it, please merge.

@FrancescAlted FrancescAlted merged commit 4132a05 into Blosc:master Jul 28, 2016
@FrancescAlted
Copy link
Member

Done!

@esc esc deleted the fixup_changelog branch August 8, 2016 19:23
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.

None yet

3 participants