We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
elastic/apm-integration-testing#1033 was required since elastic/apm-integration-testing#1016 was merged with a lint defect.
It was initially found and reported in the GitHub comment:
But the following commits didn't change those files and the preCommit didn't detect any failures.
The preCommit step cares about the current changeset, this works well as long as the following changesets are incrementally fixing the linted issues.
But it does not work well when:
Then, the preCommit won't be able to fail in the following builds since the preCommit won't detect those changesets.
As a consequence, the failure will only happen in the target branch.
preCommit should compare <target-branch>...<pr-head> rather than <target-branch>...<GIT_BASE_COMMIT
<target-branch>...<pr-head>
<target-branch>...<GIT_BASE_COMMIT
cc @elastic/observablt-robots
The text was updated successfully, but these errors were encountered:
We won't invest anymore in the existing Jenkins implementation unless a security or a major bug, hence I'll close this issue now
Sorry, something went wrong.
No branches or pull requests
What
elastic/apm-integration-testing#1033 was required since elastic/apm-integration-testing#1016 was merged with a lint defect.
It was initially found and reported in the GitHub comment:
But the following commits didn't change those files and the preCommit didn't detect any failures.
Why
The preCommit step cares about the current changeset, this works well as long as the following changesets are incrementally fixing the linted issues.
But it does not work well when:
Then, the preCommit won't be able to fail in the following builds since the preCommit won't detect those changesets.
As a consequence, the failure will only happen in the target branch.
Proposal
preCommit should compare
<target-branch>...<pr-head>
rather than<target-branch>...<GIT_BASE_COMMIT
cc @elastic/observablt-robots
The text was updated successfully, but these errors were encountered: