Skip to content
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

UP018: Improve Fix message #7913

Merged
merged 2 commits into from
Oct 11, 2023
Merged

UP018: Improve Fix message #7913

merged 2 commits into from
Oct 11, 2023

Conversation

T-256
Copy link
Contributor

@T-256 T-256 commented Oct 11, 2023

Summary

closes #7912

Test Plan

Copy link
Member

@konstin konstin left a comment

Choose a reason for hiding this comment

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

Thanks

@konstin konstin enabled auto-merge (squash) October 11, 2023 09:39
@konstin konstin merged commit a1ee6d2 into astral-sh:main Oct 11, 2023
15 checks passed
@github-actions
Copy link
Contributor

PR Check Results

Ecosystem

✅ ecosystem check detected no changes.

konstin pushed a commit that referenced this pull request Oct 11, 2023
<!--
Thank you for contributing to Ruff! To help us out with reviewing,
please consider the following:

- Does this pull request include a summary of the change? (See below.)
- Does this pull request include a descriptive title?
- Does this pull request include references to any relevant issues?
-->

## Summary

<!-- What's the purpose of the change? What does it do, and why? -->
closes #7912

## Test Plan

<!-- How was it tested? -->
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.

Confusing UP018 fix title
2 participants