Skip to content

Disable ruff rule PLW0717#863

Closed
alifbe wants to merge 1 commit into
equinor:mainfrom
alifbe:fix-ruff-0.15.14
Closed

Disable ruff rule PLW0717#863
alifbe wants to merge 1 commit into
equinor:mainfrom
alifbe:fix-ruff-0.15.14

Conversation

@alifbe
Copy link
Copy Markdown
Collaborator

@alifbe alifbe commented May 24, 2026

Disable ruff rule PLW0717 - too-many-statements-in-try-clause ()

https://docs.astral.sh/ruff/rules/too-many-statements-in-try-clause/

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.51%. Comparing base (88003f2) to head (83a41f8).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #863      +/-   ##
==========================================
+ Coverage   83.52%   84.51%   +0.98%     
==========================================
  Files          49       49              
  Lines        7279     7349      +70     
==========================================
+ Hits         6080     6211     +131     
+ Misses       1199     1138      -61     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alifbe alifbe requested review from larsevj and rnyb May 24, 2026 14:32
Copy link
Copy Markdown
Collaborator

@rnyb rnyb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Should we add an issue as a reminder to consider making code compatible with this newly added ruff rule?

@rnyb
Copy link
Copy Markdown
Collaborator

rnyb commented May 24, 2026

See #864 and #865

@alifbe
Copy link
Copy Markdown
Collaborator Author

alifbe commented May 25, 2026

@rnyb Great job in PR #865

Close this PR as it is resolved nicely in #865

@alifbe alifbe closed this May 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants