Skip to content

[pylint] Improve phrasing (PLC0208)#24033

Merged
ntBre merged 1 commit intoastral-sh:mainfrom
hunterhogan:patch-1
Mar 18, 2026
Merged

[pylint] Improve phrasing (PLC0208)#24033
ntBre merged 1 commit intoastral-sh:mainfrom
hunterhogan:patch-1

Conversation

@hunterhogan
Copy link
Contributor

Since I already put a lot of thought into the grammar and style, at least we can benefit from those changes.

Preemptive response:
I use this linter and follow its advice because I'm not an expert programmer. If you were to give me two similar Python functions, I would probably not be able to say which function was better.

On the other hand, I have exceptional training and experience as an editor and a writer. I can say whether changing one word in a draft does or does not affect the grammar, the style, the voice, or more than one.

I'm gifting to Astral the benefit of my professional knowledge by editing this text. I will not, however, attempt to explain how to write gooder and more clearer and conciser itself, which required thousands of hours to learn, in this Pull Request.

... or in future Pull Requests, if I were to choose to gift my knowledge again.

Summary

Test Plan

Since I already put a lot of thought into the grammar and style, at least we can benefit from those changes.

Preemptive response:
I use this linter and follow its advice because I'm not an expert programmer. If you were to give me two similar Python functions, I would probably not be able to say which function was better. 

On the other hand, I have exceptional training and experience as an editor and a writer. I can say whether changing one word in a draft does or does not affect the grammar, the style, the voice, or more than one.

I'm gifting to Astral the benefit of my professional knowledge by editing this text. I will not, however, attempt to explain how to write gooder and more clearer and conciser itself, which required thousands of hours to learn, in this Pull Request.

... or in future Pull Requests, if I were to choose to gift my knowledge again.
@astral-sh-bot astral-sh-bot bot requested a review from ntBre March 18, 2026 15:01
@hunterhogan
Copy link
Contributor Author

Cf. #24002

@astral-sh-bot
Copy link

astral-sh-bot bot commented Mar 18, 2026

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@AlexWaygood AlexWaygood added the documentation Improvements or additions to documentation label Mar 18, 2026
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.

Thanks!

@ntBre ntBre changed the title Correct grammar; improve style. [pylint] Improve phrasing (PLC0208) Mar 18, 2026
@ntBre ntBre merged commit b5665bd into astral-sh:main Mar 18, 2026
42 checks passed
@hunterhogan hunterhogan deleted the patch-1 branch March 19, 2026 12:17
carljm added a commit that referenced this pull request Mar 25, 2026
* main:
  [`pylint`] Improve phrasing (`PLC0208`) (#24033)
  test: migrate `show_settings` and `version` tests to use `CliTest` (#23702)
  Drain file watcher events during test setup (#24030)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants