diff options
| author | Michael Muré <batolettre@gmail.com> | 2018-12-27 21:53:48 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-12-27 21:53:48 +0100 |
| commit | 171acad6194dc1ceecca36d3d50795d5c84985d1 (patch) | |
| tree | 59ac2fd7dfdc801b434cf41478a1933a4a8423e1 /bug | |
| parent | 3f694195f86f22ac10e04bf0f0f70535fa30a4a0 (diff) | |
| parent | 11e04748b93621a7d377d3163381ff1f38e5fdbe (diff) | |
| download | git-bug-171acad6194dc1ceecca36d3d50795d5c84985d1.tar.gz | |
Merge pull request #84 from Steap/allow-empty-comment-message
Allow empty messages in comments.
Diffstat (limited to 'bug')
| -rw-r--r-- | bug/op_add_comment.go | 4 | ||||
| -rw-r--r-- | bug/operation_test.go | 1 |
2 files changed, 0 insertions, 5 deletions
diff --git a/bug/op_add_comment.go b/bug/op_add_comment.go index c19b9593..2d6fb21a 100644 --- a/bug/op_add_comment.go +++ b/bug/op_add_comment.go @@ -58,10 +58,6 @@ func (op *AddCommentOperation) Validate() error { return err } - if text.Empty(op.Message) { - return fmt.Errorf("message is empty") - } - if !text.Safe(op.Message) { return fmt.Errorf("message is not fully printable") } diff --git a/bug/operation_test.go b/bug/operation_test.go index 9296e247..255d6d98 100644 --- a/bug/operation_test.go +++ b/bug/operation_test.go @@ -47,7 +47,6 @@ func TestValidate(t *testing.T) { NewSetTitleOp(rene, unix, "title", "multi\nline"), NewSetTitleOp(rene, unix, "title\u001b", "title2"), NewSetTitleOp(rene, unix, "title", "title2\u001b"), - NewAddCommentOp(rene, unix, "", nil), NewAddCommentOp(rene, unix, "message\u001b", nil), NewAddCommentOp(rene, unix, "message", []git.Hash{git.Hash("invalid")}), NewSetStatusOp(rene, unix, 1000), |