Skip to content

Make all build.sh files executable#1202

Merged
pascalberger merged 2 commits intodevelopfrom
copilot/fix-1201
Aug 12, 2025
Merged

Make all build.sh files executable#1202
pascalberger merged 2 commits intodevelopfrom
copilot/fix-1201

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Aug 11, 2025

This PR ensures that all build.sh files across the repository are committed as executable files, providing consistency for build scripts used in integration tests.

Problem

Several build.sh files in the repository were committed without executable permissions, making them inconsistent with others and requiring users to manually add execute permissions or use bash build.sh instead of ./build.sh.

Solution

Changed file permissions from 100644 (non-executable) to 100755 (executable) for 8 build.sh files:

  • tests/Cake.Issues.GitRepository/script-runner/build.sh
  • tests/Cake.Issues.Markdownlint/script-runner/build.sh
  • tests/Cake.Issues.Reporting.Sarif/script-runner/build.sh
  • tests/Cake.Issues.Reporting.Sarif/frosting/net9/build.sh
  • tests/Cake.Issues.Reporting.Sarif/frosting/net8/build.sh
  • tests/Cake.Issues.Reporting.Generic/script-runner/build.sh
  • tests/Cake.Issues.Reporting.Generic/frosting/build.sh
  • tests/Cake.Issues.Reporting.Console/frosting/net8.0/build.sh

All 14 build.sh files in the repository now have consistent executable permissions, allowing them to be run directly with ./build.sh without requiring a bash prefix.

Testing

  • Verified the main build system continues to work correctly
  • Tested execution of newly executable build.sh files
  • Confirmed no file content was modified, only permissions

Fixes #1201.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: pascalberger <2190718+pascalberger@users.noreply.github.com>
Copilot AI changed the title [WIP] Make all build.sh files executable Make all build.sh files executable Aug 11, 2025
Copilot AI requested a review from pascalberger August 11, 2025 22:59
@pascalberger pascalberger marked this pull request as ready for review August 11, 2025 22:59
@pascalberger pascalberger requested a review from a team as a code owner August 11, 2025 22:59
@pascalberger pascalberger enabled auto-merge (squash) August 11, 2025 22:59
@pascalberger pascalberger disabled auto-merge August 11, 2025 23:23
@pascalberger pascalberger enabled auto-merge (squash) August 11, 2025 23:24
@pascalberger pascalberger disabled auto-merge August 12, 2025 17:33
@pascalberger pascalberger merged commit dcf1a9d into develop Aug 12, 2025
96 checks passed
@pascalberger pascalberger deleted the copilot/fix-1201 branch August 12, 2025 17:33
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.

Make all build.sh files executable

2 participants