^(\s{8}|\s{4})(?!try|else|doi|DOI|Warning|Access|Requests|Note)(\w*): (?!.*=|.*:)(.+)$

This matches strings in test_pvsystem.py wrongly

Deleting negative lookahead |.*:

^(\s{8}|\s{4})(?!try|else|doi|DOI|Warning|Access|Requests|Note)(\w*): (?!.*=)(.+)$

Reason: too many `default: ` strings

Exclude .*=| from negative lookahead

^(\s{8}|\s{4})(?!try|else|doi|DOI|Warning|Access|Requests|Note)(\w*): (?!.*:)(.+)$

Reason: just this line 🌝
Remaining matches are type annotations in modelchain.py and pvsystem.py

Regex: (?<spaces>\s{8}|\s{4})(?<name>\w*) : (?<type>.*) \(optional, default=(?<default>.*)\)$
Subs pattern: $1$2 : $3, default $4
Including: pvlib/**/*.py
Excluding: docs

No more "(optional, " found in .py files

(?<spaces> {8}| {4})(?<name>\w*) : (?<types>.*), default None$
$1$2 : $3, optional

(?<spaces> {8}| {4})(?<name>\w*) : None or (?<type>.*), optional$
$1$2 : $3, optional

(?<spaces> {8}| {4})(?<name>\w*) : (?<type>.*) or [nN]one, optional$
$1$2 : $3, optional

Over files: pvlib/**/*.py

…y-ls/pvlib-python into refactor-docstring-params