Qiskit / qiskit-ibm-runtime / 27547810130

87%

push

github

Update ruff config for more terse imports (#2921)

<!--
⚠️ The pull request title should be short detailed and understandable
for all.
⚠️ Also, please add a release note file using reno if the change needs
to be
  documented in the release notes.
⚠️ If your pull request fixes an open issue, please link to the issue.

✅ I have added the tests to cover my changes.
✅ I have updated the documentation accordingly.
✅ I have read the CONTRIBUTING document.
-->

### Summary

Disable the
[`split-on-trailing-comma`](https://docs.astral.sh/ruff/settings/#lint_isort_split-on-trailing-comma)
`isort` option, in order to automatically enforce keeping imports in one
line (if they fit).

This restricts a bit the flexibility (ie. developers cannot keep imports
in multiple lines if they fit in one line), but it allows for more
consistency and an easier time grepping over the codebase. Plus,
arguably we don't exercise that flexibility (there are a good number of
imports of just one item that are using multiple lines, likely due to
copy-pasting).

### Details and comments

Fixes N/A

There is also
https://docs.astral.sh/ruff/settings/#format_skip-magic-trailing-comma,
which applies not only to `import` but to the other statement. It felt a
bit aggressive to apply (dry-running it, it did caught some instances
where being more compact made sense, but there seemed to be enough
examples for extra lines included for providing more readability).

### AI/LLM disclosure

- [x] I didn't use LLM tooling, or only used it privately.

35 of 35 new or added lines in 20 files covered. (100.0%)

8080 of 9286 relevant lines covered (87.01%)

8.64 hits per line