Skip to content

[case_sensitive_file_extension_comparisons]: Don't trigger on digits-only extensions #12293

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

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

Ethiraric
Copy link
Contributor

@Ethiraric Ethiraric commented Feb 14, 2024

If we find a file extension check with only digits (.123), do not trigger case_sensitive_file_extension_comparisons.

Fixes #12252


Please write a short comment explaining your change (or "none" for internal only changes)

changelog: [case_sensitive_file_extension_comparisons]: Don't trigger on digits-only extensions

@rustbot
Copy link
Collaborator

rustbot commented Feb 14, 2024

r? @y21

rustbot has assigned @y21.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Feb 14, 2024
@GuillaumeGomez
Copy link
Member

Looks good to me, nice work!

@y21
Copy link
Member

y21 commented Feb 14, 2024

Looks good to me as well. Thanks for the nice fix!

@bors r+

@bors
Copy link
Contributor

bors commented Feb 14, 2024

📌 Commit 9492de5 has been approved by y21

It is now in the queue for this repository.

bors added a commit that referenced this pull request Feb 14, 2024
[`case_sensitive_file_extension_comparisons`]: Don't trigger on digits-only extensions

If we find a file extension check with only digits (`.123`), do not trigger `case_sensitive_file_extension_comparisons`.

Fixes #12252

---

*Please write a short comment explaining your change (or "none" for internal only changes)*

changelog: [`case_sensitive_file_extension_comparisons`]: Don't trigger on digits-only extensions
@bors
Copy link
Contributor

bors commented Feb 14, 2024

⌛ Testing commit 9492de5 with merge cf99a7a...

@bors
Copy link
Contributor

bors commented Feb 14, 2024

💔 Test failed - checks-action_test

@y21
Copy link
Member

y21 commented Feb 14, 2024

Looks like our macOS runners are still flaky :(

bors pls 🥺
@bors retry

@Ethiraric
Copy link
Contributor Author

This got me scared for a minute. The logs don't seem to contain any useful information to debug what is happening :(

@bors
Copy link
Contributor

bors commented Feb 14, 2024

⌛ Testing commit 9492de5 with merge eb300fd...

@bors
Copy link
Contributor

bors commented Feb 14, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: y21
Pushing eb300fd to master...

@bors bors merged commit eb300fd into rust-lang:master Feb 14, 2024
@Ethiraric Ethiraric deleted the fix-12252 branch February 25, 2024 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

case_sensitive_file_extension_comparisons hits on extension consisting of numbers
5 participants