-
Notifications
You must be signed in to change notification settings - Fork 9
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
Update action to exit when msg is empty #4
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the quick fix 👍
Two comment from my side, but none is blocking
github.com/pkg/errors v0.9.0 | ||
go.uber.org/zap v1.13.0 // indirect | ||
golang.org/x/crypto v0.0.0-20200109152110-61a87790db17 // indirect | ||
gopkg.in/yaml.v2 v2.2.7 // indirect |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How did you generate the file? Are the indirect imports needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i can build it again
go 1.13 | ||
|
||
require ( | ||
github.com/mattermost/mattermost-server/v5 v5.3.2-0.20200114132950-e77ef9733c97 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How did you generate this line? I was expecting it to look like https://github.com/mattermost/mattermost-plugin-jira/blob/master/go.mod#L11
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No objections beyond @hanzei's concerns.
Summary
if Text or Attachments are empty exit without failing
first part to fix mattermost/mattermost-developer-documentation#496 (comment)