Skip to content
This repository has been archived by the owner on Feb 22, 2024. It is now read-only.

Changed comment content to be required, Fixes #455 #457

Merged
merged 2 commits into from
May 29, 2017
Merged

Changed comment content to be required, Fixes #455 #457

merged 2 commits into from
May 29, 2017

Conversation

Fastjur
Copy link
Member

@Fastjur Fastjur commented May 16, 2017

Fixes #455

@jwgmeligmeyling
Copy link
Member

Could you rebase this on master so that we can see whether the build passes?

@jwgmeligmeyling jwgmeligmeyling changed the title Changed comment content to be required Changed comment content to be required, Fixes #455 May 25, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 55.993% when pulling 2d56cc1 on empty-comments into ae19a26 on master.

@Fastjur Fastjur merged commit 1c1883d into master May 29, 2017
@Fastjur Fastjur deleted the empty-comments branch May 29, 2017 13:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jwgmeligmeyling jwgmeligmeyling jwgmeligmeyling approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Fastjur @jwgmeligmeyling @coveralls