Skip to content

Conversation

@pllim
Copy link
Collaborator

@pllim pllim commented Jul 23, 2025

Closes #9668 (Originally I wanted to wait for #9520 but then I saw the problem is more widespread than a few CSV files, so here we are...)

This PR addresses unnecessary 755 mode on files when it could be 644. I spared those in scripts folder.

Note: find jwst/ -type f -executable | xargs chmod 644

Tasks

@codecov
Copy link

codecov bot commented Jul 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.44%. Comparing base (6012957) to head (076c30b).
⚠️ Report is 294 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #9690   +/-   ##
=======================================
  Coverage   80.44%   80.44%           
=======================================
  Files         367      367           
  Lines       37407    37407           
=======================================
  Hits        30093    30093           
  Misses       7314     7314           

☔ 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.

@pllim pllim marked this pull request as ready for review July 23, 2025 13:41
@pllim pllim requested a review from a team as a code owner July 23, 2025 13:41
@pllim pllim requested a review from melanieclarke July 23, 2025 13:42
@melanieclarke melanieclarke added this to the Build 12.1 milestone Jul 23, 2025
Copy link
Collaborator

@melanieclarke melanieclarke left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@melanieclarke melanieclarke merged commit ef9c5b5 into spacetelescope:main Jul 23, 2025
32 of 34 checks passed
@pllim pllim deleted the ch-ch-ch-chmod branch July 23, 2025 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment