Fix PLW1508 false positive for default string created via a mult operation

[UnknownPlatypus]

Summary

Fix a false positive for default string created via a mult operation.

MY_VAR = os.environ.get("MY_VAR", "0" * 32)

I find the if matches!(op, Operator::Mult) branch inside the Operator::Mult match quite inelegant but was not sure of the most idiomatic way to handle this.

Test Plan

cargo nextest run and cargo insta test.

[github-actions]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

[AlexWaygood]

Thanks!

[AlexWaygood]

I find the if matches!(op, Operator::Mult) branch inside the Operator::Mult match quite inelegant but was not sure of the most idiomatic way to handle this.

Agreed -- I fixed it by splitting out the Operator::Mult handling into its own branch 👍 I think that makes more sense, even if it does introduce a tiny bit of repetition.

[UnknownPlatypus]

Wonderful thanks !

Btw, just wanted to say that the dev onboarding was really simple. The Contributing.md is doing a great job.
Thanks for that.