Skip to content

[ruff] Ignore std cmath.inf (RUF069)#23120

Merged
ntBre merged 3 commits intoastral-sh:mainfrom
chirizxc:RUF069
Feb 6, 2026
Merged

[ruff] Ignore std cmath.inf (RUF069)#23120
ntBre merged 3 commits intoastral-sh:mainfrom
chirizxc:RUF069

Conversation

@chirizxc
Copy link
Contributor

@chirizxc chirizxc commented Feb 6, 2026

Summary

See: #20585

Test Plan

cargo nextest run ruf069

@chirizxc
Copy link
Contributor Author

chirizxc commented Feb 6, 2026

@ntBre It seems that this was forgotten, since the rule description mentions cmath.

@astral-sh-bot
Copy link

astral-sh-bot bot commented Feb 6, 2026

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Copy link
Contributor

@ntBre ntBre left a comment

Choose a reason for hiding this comment

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

Thank you!

@ntBre ntBre added rule Implementing or modifying a lint rule preview Related to preview mode features labels Feb 6, 2026
@ntBre ntBre merged commit 5a5d3b0 into astral-sh:main Feb 6, 2026
44 checks passed
@chirizxc
Copy link
Contributor Author

chirizxc commented Feb 6, 2026

Thank you!

Btw, have astral-sh restricted editing of PR texts for the ruff repository, or has the edit button next to the RP name simply disappeared?

изображение

@chirizxc chirizxc deleted the RUF069 branch February 6, 2026 13:42
@ntBre
Copy link
Contributor

ntBre commented Feb 6, 2026

I think the edit button goes away once a PR is merged. At least I see the same thing on some of my old PRs outside of Ruff. We haven't changed any settings as far as I know!

I can still see the button here as a maintainer. Did you want me to update the title?

@chirizxc
Copy link
Contributor Author

chirizxc commented Feb 6, 2026

I think the edit button goes away once a PR is merged. At least I see the same thing on some of my old PRs outside of Ruff. We haven't changed any settings as far as I know!

I can still see the button here as a maintainer. Did you want me to update the title?

This was before the merger, I wanted to change it, since the change also affected cmath.infj.

@chirizxc
Copy link
Contributor Author

chirizxc commented Feb 6, 2026

I can still see the button here as a maintainer. Did you want me to update the title?

I don't really care, but just for future reference, if I open a PR with the title: [WIP] ..., then I won't be able to edit it myself, which is quite strange.

@chirizxc
Copy link
Contributor Author

chirizxc commented Feb 6, 2026

#22560

изображение

I don't have an Edit button here either.

@ntBre
Copy link
Contributor

ntBre commented Feb 6, 2026

Oh, that is strange. Hopefully it's a temporary GitHub bug and not a new change!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview Related to preview mode features rule Implementing or modifying a lint rule

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants