refactor(bump): eliminate similar patterns in code

[bearomorphism]

Description

Checklist

• I have read the contributing guidelines

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.90%. Comparing base (a9cd957) to head (99bd410).
Report is 13 commits behind head on refactors.

Additional details and impacted files

@@              Coverage Diff              @@
##           refactors    #1508      +/-   ##
=============================================
+ Coverage      97.87%   97.90%   +0.03%     
=============================================
  Files             57       57              
  Lines           2677     2673       -4     
=============================================
- Hits            2620     2617       -3     
+ Misses            57       56       -1     

Flag	Coverage Δ
unittests	97.90% <100.00%> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Lee-W]

Suggested change

	(out.diagnostic if self.git_output_to_stderr else out.write)(msg)
	out_func_name = "diagnostic" if self.git_output_to_stderr else "write"
	getattr(out, out_func_name)(msg)

This might be a bit easier to read 🤔

[bearomorphism]

iirc getattr does not have good type annotation. I prefer the following and still easy to read.

                out_func = out.diagnostic if self.git_output_to_stderr else out.write

[bearomorphism]

fixed.