Link to issue number:

In-response-to #12355 (comment)

Summary of the issue:

Pre-commit-ci, on behalf of Ruff, does not approve of function parameter lists being double indented.
I.e. NVDA's coding standards said:

For the parameter list of function definitions, double indent, this differentiates the parameters and the body of the function.

That resulted in, e.g.:

def test(
		param1: str,
		param2: int,
		param3: bool = False,
) -> str:
	pass

Pushing a function definition like that to an NVDA PR, would result in Pre-commit-ci "fixing" it, to appear as:

def test(
	param1: str,
	param2: int,
	param3: bool = False,
) -> str:
	pass

Sean suggested updating coding standards, which is presumably easier than trying to convince Ruff to do it our way.

Description of user facing changes

None

Description of development approach

• Updated coding standards.
• Also noticed that there was no instruction on how to format script names, but there was for events and various other entities. Added one to the relevant section.
• Changed a few "eg"s to "e.g."s, and made other slight grammar alterations.