Skip to content
This repository has been archived by the owner on Nov 8, 2018. It is now read-only.

update changelog for 0.4.4 #1477

Merged
merged 1 commit into from
May 6, 2016
Merged

update changelog for 0.4.4 #1477

merged 1 commit into from
May 6, 2016

Conversation

ChristophWurst
Copy link
Contributor

I updated the changelog following advices on http://keepachangelog.com/.

We should add a changelog entry for EVERY feature/fix/whatever

@ChristophWurst ChristophWurst added this to the 0.4.4 milestone May 5, 2016
@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @Gomez and @tahaalibra to be potential reviewers

@ChristophWurst
Copy link
Contributor Author

Btw. do we need to mention the developer of the PR? To be fair we should either also mention the one who discovered/reported the bug as this is often more work to do than fixing it, or don't mention anyone at all. Personally, I prefer the latter.

@Gomez
Copy link
Contributor

Gomez commented May 6, 2016

Ok, lets drop the name.
The ### Added, ### Changed, ### Fixed is more work. Is it worth it?

I will merge an we think about it until the next rls...

@Gomez Gomez merged commit b356f20 into master May 6, 2016
@Gomez Gomez deleted the changelog044 branch May 6, 2016 09:08
@ChristophWurst
Copy link
Contributor Author

Is it worth it?

I think so, yes ;-)

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