chore: cleanup unnecessary B009 ignore

syrupy-project / syrupy

:pancakes: The sweeter pytest snapshot plugin

https://syrupy-project.github.io/syrupy/

Apache License 2.0

550 stars 36 forks source link

chore: cleanup unnecessary B009 ignore #915

Closed epenet closed 1 week ago

epenet commented 2 weeks ago

Description

I can't think of a good reason to keep using getattr It made sense when there was a default None - but now that there isn't a default it doesn't make sense...

Related Issues

See #583 where it was added

Checklist

[ ] This PR has sufficient documentation.
[ ] This PR has sufficient test coverage.
[x] This PR title satisfies semantic convention.

Additional Comments

No additional comments.

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

@@           Coverage Diff           @@
##             main     #915   +/-   ##
=======================================
  Coverage   97.74%   97.74%           
=======================================
  Files          21       21           
  Lines        1596     1596           
=======================================
  Hits         1560     1560           
  Misses         36       36

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests
JS Bundle Analysis - Avoid shipping oversized bundles

noahnu commented 8 hours ago

:tada: This PR is included in version 4.8.0 :tada:

The release is available on:

v4.8.0
GitHub release

Your semantic-release bot :package::rocket: