Closed hamelsmu closed 4 years ago
test of input: npi.pass
All Checkpoints for: 463e2e4d34dfcd5a32af5f1ab45c47c15bdceed8 have passed. Corresponding Data Docs have been generated and can be viewed here.
test of input: npi.fail
Failed Great Expectations checkpoint(s) npi.fail
detected for: 463e2e4d34dfcd5a32af5f1ab45c47c15bdceed8. Corresponding Data Docs have been generated and can be viewed here.
test of input: npi.pass,npi.fail
Failed Great Expectations checkpoint(s) npi.fail
detected for: 463e2e4d34dfcd5a32af5f1ab45c47c15bdceed8. Corresponding Data Docs have been generated and can be viewed here.
test of input: npi.fail,npi.pass
Failed Great Expectations checkpoint(s) npi.fail
detected for: 463e2e4d34dfcd5a32af5f1ab45c47c15bdceed8. Corresponding Data Docs have been generated and can be viewed here.
Ok lets see if commenting works as well.
/data-docs
test of input (PR Comment): npi.fail
Failed Great Expectations checkpoint(s) npi.fail
detected for: a77ea1c5ed26d82f4dea8d87f14ddd8c05824818. Corresponding Data Docs have been generated and can be viewed here.
test of input (PR Comment): npi.pass,npi.fail
Failed Great Expectations checkpoint(s) npi.fail
detected for: a77ea1c5ed26d82f4dea8d87f14ddd8c05824818. Corresponding Data Docs have been generated and can be viewed here.
test of input (PR Comment): npi.fail,npi.pass
Failed Great Expectations checkpoint(s) npi.fail
detected for: a77ea1c5ed26d82f4dea8d87f14ddd8c05824818. Corresponding Data Docs have been generated and can be viewed here.
test of input (PR Comment): npi.pass
All Checkpoints for: a77ea1c5ed26d82f4dea8d87f14ddd8c05824818 have passed. Corresponding Data Docs have been generated and can be viewed here.
@Aylr see above. Tested for both comments and for PR activity.
For all practical purposes, I only think Actions should comment on pushes when there is a failure, otherwise would be way too much noise for most people to just automatically comment on each success.
However, for the chat command, I think it should comment either way on success OR failure. WDYT? I have updated the README accordingly.
I believe we are all good.
This and prior PRs close #71
I'm testing the Action Removing badges for tests that fail on purpose because we are testing the test
cc: @Aylr