- Issue created by @longwave
- Merge request !6449Use CI_MERGE_REQUEST_TARGET_BRANCH_SHA or CI_MERGE_REQUEST_DIFF_BASE_SHA for test-only changes. β (Open) created by longwave
- Status changed to Needs review
11 months ago 9:26am 5 February 2024 - Status changed to RTBC
11 months ago 2:56pm 9 February 2024 - πΊπΈUnited States smustgrave
Have 100% seen this problem, as have many others.
Not sure how to test but the change appears to make sense. May be one of those issues won't know it works till it's used live a few times. So maybe we could merge on a Monday to keep an eye through the week?
- π¬π§United Kingdom longwave UK
As it's the test-only feature which isn't critical I think it could be committed any time, happy to revisit if we still find issues after this.
- Status changed to Fixed
10 months ago 5:34pm 20 February 2024 - π¬π§United Kingdom catch
Committed/pushed to 11.x and cherry-picked to 10.2.x, thanks!
- πΊπΈUnited States benjifisher Boston area
smustgrave β credited benjifisher β .
- πΊπΈUnited States smustgrave
Closed π The test-only script for GitLab CI does not reliably use the right commit Needs review as a duplicate
Automatically closed - issue fixed for 2 weeks with no activity.