Allow additional arguments for sum and max comprehensions

[charliermarsh]

Summary

These can have other arguments, so it seems wrong to gate on single argument here.

Closes #12358.

[charliermarsh]

Without this, we were (1) dropping commas in some tests (see fixture changes), and (2) adding an extra space after the comma when fixing sum([x.val for x in bar], 0) (not sure why?).

[dhruvmanila]

Hmm, I don't remember why this was None but this seems correct to me. It might've been some weird comment placement, not sure though.

[github-actions]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

ℹ️ ecosystem check detected linter changes. (+6 -0 violations, +0 -0 fixes in 2 projects; 52 projects unchanged)

bokeh/bokeh (+1 -0 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --ignore RUF9 --output-format concise --preview --select ALL

+ examples/models/calendars.py:47:31: C419 Unnecessary list comprehension

mlflow/mlflow (+5 -0 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --ignore RUF9 --output-format concise --preview

+ tests/store/tracking/__init__.py:62:27: C419 Unnecessary list comprehension
+ tests/store/tracking/test_file_store.py:372:23: C419 Unnecessary list comprehension
+ tests/store/tracking/test_sqlalchemy_store.py:2548:28: C419 Unnecessary list comprehension
+ tests/store/tracking/test_sqlalchemy_store.py:2562:28: C419 Unnecessary list comprehension
+ tests/store/tracking/test_sqlalchemy_store.py:282:23: C419 Unnecessary list comprehension

Changes by rule (1 rules affected)

code	total	+ violation	- violation	+ fix	- fix
C419	6	6	0	0	0

[dhruvmanila]

Good find!