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
Add timestamp/SHA to PR comment #1708
Conversation
Downstream Build Status ReportBuild
|
Pull Request Test Coverage Report for Build 8410579111Details
💛 - Coveralls |
Unfortunately we're not seeing what the comment will look like after this PR is merged. I think it might be because of how Downstream Build Status Report - 10ba101 |
I this meant to add a timestamp as well? or just a commit hash? |
I initially had it add a timestamp too but figured the commit hash was more relevant. Forgot to change the PR title. I can add the timestamp back in if you'd like me to. |
I think both are useful since it can often be easier to check the timestamp without cross-referencing |
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 @bennibbelink - let's give this a shot
Closes issue #1707. Adds a timestamp and commit hash to the PR comment generated on downstream testing