Skip to content

Set executable bit on jqschema.sh to unblock Copilot PR data fetch#34301

Merged
pelikhan merged 1 commit into
mainfrom
copilot/fix-permission-on-jqschema
May 23, 2026
Merged

Set executable bit on jqschema.sh to unblock Copilot PR data fetch#34301
pelikhan merged 1 commit into
mainfrom
copilot/fix-permission-on-jqschema

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 23, 2026

The Copilot PR Prompt Pattern Analysis workflow fails in Fetch Copilot PR data because .github/skills/jqschema/jqschema.sh is invoked directly but is not executable. This change aligns repository file mode with runtime usage.

  • Root cause

    • jqschema.sh was committed with mode 100644, but workflow steps execute it as ./.github/skills/jqschema/jqschema.sh.
  • Change

    • Update file mode for .github/skills/jqschema/jqschema.sh from 100644 to 100755.
  • Impact

    • Removes Permission denied at the jqschema invocation point in the agent job, allowing PR data schema generation to proceed.
# before
100644 .github/skills/jqschema/jqschema.sh

# after
100755 .github/skills/jqschema/jqschema.sh

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title Fix jqschema script permission in workflows Set executable bit on jqschema.sh to unblock Copilot PR data fetch May 23, 2026
Copilot AI requested a review from pelikhan May 23, 2026 21:17
@pelikhan pelikhan marked this pull request as ready for review May 23, 2026 21:23
Copilot AI review requested due to automatic review settings May 23, 2026 21:23
@pelikhan pelikhan merged commit f8aab9f into main May 23, 2026
@pelikhan pelikhan deleted the copilot/fix-permission-on-jqschema branch May 23, 2026 21:23
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

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