-
Notifications
You must be signed in to change notification settings - Fork 38.8k
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
Automated cherry pick of #71923: diff: Fix overlapping filenames #71955
Automated cherry pick of #71923: diff: Fix overlapping filenames #71955
Conversation
/sig cli |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: apelisse The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/lgtm |
/retest |
/hold We should use a file name that is safe cross-platform See |
The filename can overlap when multiple resources have the same name (but obviously are of a different type). Include the name of the type in the file name to prevent the overlap.
9c8279b
to
4980403
Compare
/hold cancel |
/lgtm thanks |
/retest |
@apelisse the bot's not picking up release notes apparently when there are multiple PR's commites being cherry picked. Can you add a release note to this PR's description text? |
Cherry pick of #71923 and #71974 on release-1.13.
#71923: diff: Fix overlapping filenames
#71974