Skip to content

[flake8-async] Add fix safety section (ASYNC116) #17497

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.

Sign up for GitHub

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

Jump to bottom
Merged
dylwil3 merged 4 commits into from
Apr 26, 2025
Merged

[flake8-async] Add fix safety section (ASYNC116) #17497

dylwil3 merged 4 commits into from
Apr 26, 2025

Conversation

@Kalmaegi
Copy link
Contributor

@Kalmaegi Kalmaegi commented Apr 20, 2025 •
edited
Loading

Summary

This PR add the fix safety section for rule ASYNC116 in long_sleep_not_forever.rs for #15584

@Kalmaegi Kalmaegi changed the title [flake8-async] add fix safety section (ASYNC116) [flake8-async] add fix safety section (ASYNC116) Apr 20, 2025
@Kalmaegi Kalmaegi changed the title [flake8-async] add fix safety section (ASYNC116) [flake8-async] Add fix safety section (ASYNC116) Apr 20, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Apr 20, 2025 •
edited
Loading

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@dhruvmanila dhruvmanila added the documentation Improvements or additions to documentation label Apr 21, 2025
@dylwil3 dylwil3 self-assigned this Apr 22, 2025
dylwil3
dylwil3 approved these changes Apr 26, 2025
View reviewed changes
Copy link
Collaborator

@dylwil3 dylwil3 left a comment

Choose a reason for hiding this comment

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

Thanks! I think we can be more concise here.

@dylwil3 dylwil3 merged commit a4e225e into astral-sh:main Apr 26, 2025
33 checks passed
@BrewTestBot BrewTestBot mentioned this pull request May 1, 2025
Merged
@MeGaGiGaGon MeGaGiGaGon mentioned this pull request Jun 21, 2025
Open
71 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dylwil3 dylwil3 dylwil3 approved these changes

Assignees

@dylwil3 dylwil3

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Kalmaegi @dylwil3 @dhruvmanila