Open valscion opened 8 months ago
I wonder if the call in here is missing the last optional argument depicting what the commit hash is?
Looking at what this code could use:
EDIT: Looking at the codepath this seems to be correct for us. We are not using inline comments but are instead in favor of posting a single comment that Danger keeps up-to-date and using the commit status checks here.
The fix looks fairly trivial. I can try doing one if you so prefer.
@valscion We’re always happy to review PRs fixing issues. This is a collaborative open-source project!
Describe the bug
Danger associates the commit status check with the latest commit in an open GitHub pull request when running in CircleCI.
This is problematic when the pull request has more Danger runs and the later Danger run completes faster than the original Danger run. This makes the entire pull request status checks red if Danger has failures.
The commit where Danger did complain on purpose had these environment variables:
The commit where Danger didn't complain but which completed sooner than the one that did complain:
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Danger should associate the GitHub commit status with the commit that was failing and not with the latest commit in the PR.
If possible, Danger should use the
CIRCLE_SHA1
environment variable to associate the status check with the correct commit.Screenshots
Your Environment
Additional context
N/A