Skip to content

Fix: Add execute permissions to validation scripts#2902

Open
vijaysaayi wants to merge 5 commits intomainfrom
user/visaayir/fix-validate-sha-permissions
Open

Fix: Add execute permissions to validation scripts#2902
vijaysaayi wants to merge 5 commits intomainfrom
user/visaayir/fix-validate-sha-permissions

Conversation

@vijaysaayi
Copy link
Copy Markdown
Contributor

@vijaysaayi vijaysaayi commented Apr 10, 2026 •
edited
Loading

  • The purpose of this PR is explained in this message or in an issue. If an issue please include a reference as #<issue_number>.
  • Tests are included and/or updated for code changes.
  • Proper license headers are included in each file.

Add execute permissions to validation scripts

Vijay Saayi Ravinuthala and others added 2 commits April 10, 2026 15:19
Fix: Add execute permissions to validation scripts
dc0c040 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
TEST: Invalid SHA to verify validation fails (revert this)
9c46e44 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@vijaysaayi vijaysaayi requested a review from a team as a code owner April 10, 2026 09:50
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

SHA Validation Summary

Stack Status
PHP ❌ failure
.NET ⏭️ skipped

Vijay Saayi Ravinuthala and others added 2 commits April 10, 2026 15:28
Improve PR comment with detailed validation output
2bd658b 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Fix: Preserve newlines in validation output
ad48ebd 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

SHA Validation Summary

Stack Status
PHP ❌ failure
.NET ✅ success
PHP Validation Output 
Validating PHP SHA256 hashes from: images/constants.yml
==================================================
PHP 8.1.34: OK
PHP 8.2.30: OK
PHP 8.3.29: OK
PHP 8.4.16: FAILED
  Expected: f66f8f48db34e9e29f7bfd6901178e9cf4a1b163e6e497716dfcb8f88bcfae30
  Got:      0000000000000000000000000000000000000000000000000000000000000000
PHP 8.5.1: OK
==================================================
Validated: 4 PHP version(s)
VALIDATION FAILED - SHA256 mismatches detected!
.NET Validation Output 
Validating .NET SHA512 hashes...
  .NET Runtime 8.0.25: OK
  .NET Runtime 9.0.14: OK
  NET_CORE_APP_100: SKIP (no SHA defined)
  ASP.NET Core 8.0.25: OK
  ASP.NET Core 9.0.14: OK
  ASPNET_CORE_APP_100: SKIP (no SHA defined)
PASSED

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

SHA Validation Summary

Stack Status
PHP ❌ failure
.NET ✅ success
PHP Validation Output 
Validating PHP SHA256 hashes from: images/constants.yml
==================================================
PHP 8.1.34: OK
PHP 8.2.30: OK
PHP 8.3.29: OK
PHP 8.4.16: FAILED
  Expected: f66f8f48db34e9e29f7bfd6901178e9cf4a1b163e6e497716dfcb8f88bcfae30
  Got:      0000000000000000000000000000000000000000000000000000000000000000
PHP 8.5.1: OK
==================================================
Validated: 4 PHP version(s)
VALIDATION FAILED - SHA256 mismatches detected!
.NET Validation Output 
Validating .NET SHA512 hashes...
  .NET Runtime 8.0.25: OK
  .NET Runtime 9.0.14: OK
  NET_CORE_APP_100: SKIP (no SHA defined)
  ASP.NET Core 8.0.25: OK
  ASP.NET Core 9.0.14: OK
  ASPNET_CORE_APP_100: SKIP (no SHA defined)
PASSED

Revert test: Restore correct PHP 8.4 SHA
8283bd9 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sarsharma sarsharma sarsharma approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vijaysaayi @sarsharma