Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Update example for PT001 as per the new default behavior #13019

Merged
merged 1 commit into from
Aug 21, 2024
Merged

Update example for PT001 as per the new default behavior #13019

merged 1 commit into from
Aug 21, 2024

Conversation

FinchPowers
Copy link
Contributor

Summary

Example / Use instead were not updated with the release of ruff 0.6.0. This updates them accordingly.

Test Plan

None.

@FinchPowers
Fix docs rule PT001
5adcfdd 
Example / Use instead have proper content.
@FinchPowers FinchPowers mentioned this pull request Aug 21, 2024
Merged
@github-actions GitHub Actions
Copy link
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

dhruvmanila
dhruvmanila approved these changes Aug 21, 2024
View reviewed changes
Copy link
Member

@dhruvmanila dhruvmanila left a comment

Choose a reason for hiding this comment

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

Thanks

@dhruvmanila dhruvmanila added the documentation Improvements or additions to documentation label Aug 21, 2024
@dhruvmanila dhruvmanila changed the title Fix docs rule PT001 Update example for PT001 as per the new default behavior Aug 21, 2024
@dhruvmanila dhruvmanila merged commit dedefd7 into astral-sh:main Aug 21, 2024
20 checks passed
@BrewTestBot BrewTestBot mentioned this pull request Aug 22, 2024
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@dhruvmanila dhruvmanila dhruvmanila approved these changes

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@FinchPowers @dhruvmanila