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
Disallow empty command submission #3270
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.
I'd be surprised that this passes as is, because we have many old-style integration tests that never send actual individual commands (see MockMessages
).
c736cba
to
580d670
Compare
580d670
to
edc50de
Compare
Indeed. I changed all such tests with additional commits. Let me know if this LGTY. 🙂 |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Fixes #592
Pull Request Checklist
NOTE: CI is not automatically run on non-members pull-requests for security
reasons. The reviewer will have to comment with
/AzurePipelines run
totrigger the build.