Skip to content

Remove superfluous executable bits and shebangs#12540

Merged
berland merged 3 commits intoequinor:mainfrom
berland:ruff_exe
Dec 22, 2025
Merged

Remove superfluous executable bits and shebangs#12540
berland merged 3 commits intoequinor:mainfrom
berland:ruff_exe

Conversation

@berland
Copy link
Contributor

@berland berland commented Dec 22, 2025

Issue
Resolves ruff EXE issues.

Approach

  • PR title captures the intent of the changes, and is fitting for release notes.
  • Added appropriate release note label
  • Commit history is consistent and clean, in line with the contribution guidelines.
  • Make sure unit tests pass locally after every commit (git rebase -i main --exec 'just rapid-tests')

When applicable

  • When there are user facing changes: Updated documentation
  • New behavior or changes to existing untested code: Ensured that unit tests are added (See Ground Rules).
  • Large PR: Prepare changes in small commits for more convenient review
  • Bug fix: Add regression test for the bug
  • Bug fix: Add backport label to latest release (format: 'backport release-branch-name')

These files are imported by other scripts and should not themselves be
executable.

See also ruff rule EXE002
That line is doing nothing when the execute bit is not set.

See ruff rule EXE001.
@berland berland added this to SCOUT Dec 22, 2025
@berland berland changed the title Remove superfluous executable bist and shebangs Remove superfluous executable bits and shebangs Dec 22, 2025
@codecov-commenter
Copy link

codecov-commenter commented Dec 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.66%. Comparing base (c1021bf) to head (0d7d307).
⚠️ Report is 11 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12540      +/-   ##
==========================================
- Coverage   90.67%   90.66%   -0.02%     
==========================================
  Files         431      431              
  Lines       29726    29734       +8     
==========================================
+ Hits        26955    26959       +4     
- Misses       2771     2775       +4     
Flag Coverage Δ
cli-tests 37.62% <ø> (+<0.01%) ⬆️
gui-tests 68.75% <ø> (+<0.01%) ⬆️
performance-and-unit-tests 74.17% <ø> (+<0.01%) ⬆️
test 38.37% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@berland berland moved this to Ready for Review in SCOUT Dec 22, 2025
@berland berland self-assigned this Dec 22, 2025
@berland berland added the release-notes:maintenance Automatically categorise as maintenance change in release notes label Dec 22, 2025
Copy link
Contributor

@andreas-el andreas-el left a comment

Choose a reason for hiding this comment

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

💯

@github-project-automation github-project-automation bot moved this from Ready for Review to Reviewed in SCOUT Dec 22, 2025
@berland berland merged commit 0bac71b into equinor:main Dec 22, 2025
35 checks passed
@github-project-automation github-project-automation bot moved this from Reviewed to Done in SCOUT Dec 22, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Dec 22, 2025

CodSpeed Performance Report

Merging #12540 will not alter performance

Comparing berland:ruff_exe (0d7d307) with main (2f38385)

Summary

✅ 22 untouched

@berland berland deleted the ruff_exe branch March 6, 2026 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes:maintenance Automatically categorise as maintenance change in release notes

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants