Bump Ruff to v0.15.18 and address new lint violations

[julianz-]

Updates Ruff from v0.13.3 to v0.15.18
This requires the following changes:

• Fix D421: reword property docstrings to noun phrases
• Fix ISC004: parenthesize implicit string concatenation in collections; use single strings with # noqa: LN001 where readability is better
• Drop now-redundant # noqa: D401 comments on property definitions
• Suppress PLW0717, RUF067, S607 pending future fixes

❓ What kind of change does this PR introduce?

• 🐞 bug fix
• 🐣 feature
• 📋 docs update
• 📋 tests/coverage improvement
• 📋 refactoring
• 💥 other

📋 What is the related issue number (starting with #)

Resolves #

❓ What is the current behavior? (You can also link to an open issue here)

❓ What is the new behavior (if this is a feature change)?

📋 Other information:

📋 Contribution checklist:

(If you're a first-timer, check out
[this guide on making great pull requests][making a lovely PR])

• I wrote descriptive pull request text above
• I think the code is well written
• I wrote [good commit messages]
• I have [squashed related commits together][related squash] after
  the changes have been approved
• Unit tests for the changes exist
• Integration tests for the changes exist (if applicable)
• I used the same coding conventions as the rest of the project
• The new code doesn't generate linter offenses
• Documentation reflects the changes
• The PR relates to only one subject with a clear title
  and description in grammatically correct, complete sentences

[read-the-docs-community]

Documentation build overview

📚 cheroot | 🛠️ Build #33277062 | 📁 Comparing a9282c0 against latest (18f4ba8)

  🔍 Preview build  

5 files changed · ± 5 modified

± Modified

• history/index.html
• pkg/cheroot.connections/index.html
• pkg/cheroot.server/index.html
• pkg/cheroot.test.webtest/index.html
• pkg/cheroot.workers.threadpool/index.html

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.17%. Comparing base (18f4ba8) to head (a9282c0).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #826      +/-   ##
==========================================
- Coverage   78.19%   78.17%   -0.03%     
==========================================
  Files          41       41              
  Lines        4788     4788              
  Branches      547      547              
==========================================
- Hits         3744     3743       -1     
- Misses        905      906       +1     
  Partials      139      139              

[webknjaz]

Do you feel strongly about those noqas?

[webknjaz]

Suggested change

[webknjaz]

Are there many of these? I mostly prefer narrowly scoped ignores when adding new rules.

[julianz-]

There are 8 violations across 6 files. I've moved PLW0717 to per-file ignores (production files + test files) rather than keeping it in the global suppress list — that way the rule stays active everywhere else. Happy to switch to individual # noqa: PLW0717 lines on each violation if you prefer?

[webknjaz]

I'd like to keep these multi-line instead of having ignores. Let's go for nested parentheses instead.

[webknjaz]

You probably want this

Suggested change

	-- by :user:`julianz-`.
	-- by :user:`julianz-`

[webknjaz]

IIRC, you have to add an empty line for the RST parser to see this as a list.

Suggested change

	Bumped Ruff to v0.15.18, which introduces new rules:
	Bumped Ruff to v0.15.18, which introduces new rules:

[webknjaz]

Render's broken here: https://cheroot--826.org.readthedocs.build/en/826/history/#contributor-facing-changes

[julianz-]

Think fixed now hopefully.