Skip to content

ci: flag copied and renamed manifests for new crate check#8456

Merged
t-nelson merged 1 commit intoanza-xyz:masterfrom
t-nelson:cccr
Oct 14, 2025
Merged

ci: flag copied and renamed manifests for new crate check#8456
t-nelson merged 1 commit intoanza-xyz:masterfrom
t-nelson:cccr

Conversation

@t-nelson
Copy link
Copy Markdown

Problem

#6782 moved a cargo manifest while renaming the crate. this series of actions was not flagged by ci/check-crates.sh, allowing the pr to be merged without first reserving the new name on crates.io

Summary of Changes

adds copied and renamed cargo manifests to crate check consideration

@t-nelson t-nelson requested a review from yihau October 13, 2025 19:21
@steviez steviez requested a review from a team October 13, 2025 19:36
@t-nelson
Copy link
Copy Markdown
Author

this change probably didn't need to trigger full ci 😅

Copy link
Copy Markdown
Member

@yihau yihau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🫡

Copy link
Copy Markdown

@steviez steviez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, had to double check the help but seem correct:

$ git diff --help | grep diff-filter -A 2 
       --diff-filter=[(A|C|D|M|R|T|U|X|B)...[*]]
           Select only files that are Added (A), Copied (C), Deleted (D),
           Modified (M), Renamed (R), have their type (i.e. regular file,

@t-nelson t-nelson added this pull request to the merge queue Oct 14, 2025
Merged via the queue into anza-xyz:master with commit 401ddc2 Oct 14, 2025
33 checks passed
@t-nelson t-nelson deleted the cccr branch October 14, 2025 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants