Skip to content

chore(rustfix): rename Readme.md to README.md#13181

Merged
bors merged 2 commits into
rust-lang:masterfrom
0xPoe:rustin-patch-rename-files
Dec 18, 2023
Merged

chore(rustfix): rename Readme.md to README.md#13181
bors merged 2 commits into
rust-lang:masterfrom
0xPoe:rustin-patch-rename-files

Conversation

@0xPoe

@0xPoe 0xPoe commented Dec 18, 2023

Copy link
Copy Markdown
Member

What does this PR try to resolve?

I found it when I read the rustfix code. Usually, we use README and CHANGELOG.

How should we test and review this PR?

Check the file names.

Additional information

None

Signed-off-by: hi-rustin <rustin.liu@gmail.com>
Signed-off-by: hi-rustin <rustin.liu@gmail.com>
@rustbot

rustbot commented Dec 18, 2023

Copy link
Copy Markdown
Collaborator

r? @weihanglo

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added Command-fix S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 18, 2023

@weihanglo weihanglo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not sure how much it is relevant to be consistent, but would this cause problems on case-insensitive file systems?

@0xPoe

0xPoe commented Dec 18, 2023

Copy link
Copy Markdown
Member Author

Not sure how much it is relevant to be consistent, but would this cause problems on case-insensitive file systems?

What kind of problems? I tried to cherry-pick the commit to my latest master branch. Git can help us to rename it correctly. (I am using MacOS)

I used git mv Readme.md README.md. So I guess this is a safe change.

@0xPoe 0xPoe requested a review from weihanglo December 18, 2023 14:34
@weihanglo

Copy link
Copy Markdown
Member

Something like this. But yeah, it shouldn't matter that much and we know how to "fix it".

@bors r+

@bors

bors commented Dec 18, 2023

Copy link
Copy Markdown
Contributor

📌 Commit efdaf4b has been approved by weihanglo

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 18, 2023
@bors

bors commented Dec 18, 2023

Copy link
Copy Markdown
Contributor

⌛ Testing commit efdaf4b with merge 9c1316f...

@bors

bors commented Dec 18, 2023

Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-actions
Approved by: weihanglo
Pushing 9c1316f to master...

@bors bors merged commit 9c1316f into rust-lang:master Dec 18, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 22, 2023
Update cargo

10 commits in 1a2666ddd14cf0a255d4ddb61c63531c259a7b39..363a2d11320faf531f6aacd1ea067c6bc08343b9
2023-12-17 17:53:53 +0000 to 2023-12-22 03:12:42 +0000
- refactor: centralize git checkouts and db paths (rust-lang/cargo#13187)
- Bump to 0.78.0; update changelog (rust-lang/cargo#13192)
- refactor: custom error types for `cargo-util-schemas` (rust-lang/cargo#13186)
- chore(deps): update rust crate handlebars to `v4.5.0` (rust-lang/cargo#13168)
- Hold the mutate exclusive lock when vendoring (rust-lang/cargo#12509)
- refactor: clean up package metadata (rust-lang/cargo#13184)
- ci: check SemVer for cargo-util-schemas on CI (rust-lang/cargo#13185)
- refactor(schemas): Pull out as `cargo-util-schemas` (rust-lang/cargo#13178)
- chore(rustfix): rename Readme.md to README.md (rust-lang/cargo#13181)
- chore(rustfix): remove useless clippy rules and fix a typo (rust-lang/cargo#13182)

r? ghost
@rustbot rustbot added this to the 1.77.0 milestone Dec 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Command-fix S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants