Moving the unsupported-assignment-operation lint rule

[joesho112358]

Summary

#9614 Moving up and comment unsupported-assignment-operation

Details and comments

This looks like it should be fixed if/when it gets updated.

[qiskit-bot]

Thank you for opening a new pull request.

Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient.

While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone.

One or more of the the following people are requested to review this:

• @Qiskit/terra-core

[coveralls]

Pull Request Test Coverage Report for Build 9026416755

Details

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• 31 unchanged lines in 2 files lost coverage.
• Overall coverage decreased (-0.04%) to 89.611%

---

Files with Coverage Reduction	New Missed Lines	%
crates/qasm2/src/lex.rs	7	91.86%
crates/qasm2/src/parse.rs	24	96.23%

Totals
Change from base Build 9023909750:	-0.04%
Covered Lines:	62187
Relevant Lines:	69397

---

💛 - Coveralls

[jakelishman]

We can put a # pylint: disable=unsupported-assignment-operation with a comment on the offending line, if it's just one place.

We might even be able to get away with not doing that if we put a manual type hint

class Options(Mapping):
    __slots__ = ("_fields", "validator")
    _fields: dict[str, Any]
    validator: dict[str, Any]

in the class body - with any luck, that should get pylint to infer the proper types of those fields, even though it can't reason about the super().__setattr__ stuff we do.

[joesho112358]

the typing was unfortunately insufficient to convince the linter. updates for the single line were added

[jakelishman]

Ah, that's a shame. The skip is fine, though - it's not uncommon to have very specific lines of code where it makes sense to disable one particular linter rule, which is why the skip syntax is there.

[jakelishman]

Thanks!