Fix new instrumentation lock files leaking into core dependency update PR.#11676
Conversation
|
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
| git ls-files --others --exclude-standard -- \ | ||
| 'dd-smoke-tests/*gradle.lockfile' 'dd-java-agent/instrumentation/*gradle.lockfile' \ | ||
| | xargs -r rm -f |
There was a problem hiding this comment.
🎯 suggestion: What about git clean -f?
It can take pathspec as argument (ref).
|
/merge -f --reason "This is GitHub Action, no need to run on MQ" |
|
View all feedbacks in Devflow UI.
The expected merge time in
Warning This change was merged without running any pre merge CI checks Reason: This is GitHub Action, no need to run on MQ |
139a1ba
into
master
What Does This Do
Fixes the
update-gradle-dependenciesworkflow so a newly generated instrumentationgradle.lockfileno longer leaks into the core PR.Motivation
Correct PRs.
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issue/merge. You can also:/merge --commit-message "..."/merge -c/merge -f --reason "reason"; please use this judiciously, as some checks do not run at the PR-level (note: the PR still needs to be mergeable, this will only skip the pre-merge build)Jira ticket: [PROJ-IDENT]