fix: scope continue-on-error to fork PRs only in comment step

[Copilot]

continue-on-error: true on the PR comment step silently swallows all failures — including same-repo PRs where errors indicate real problems (auth misconfiguration, rate limiting, API changes).

Change

Make continue-on-error conditional on whether the PR originates from a fork:

- name: Post or update PR comment
  continue-on-error: ${{ github.event.pull_request.head.repo.fork == true }}  # GITHUB_TOKEN is read-only for fork PRs (dotnet/arcade-skills#13)

Fork PRs still tolerate the expected HTTP 403 (read-only GITHUB_TOKEN). Same-repo PRs now fail the job on unexpected comment errors.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>feedback on #15</issue_title>
<issue_description>address "continue-on-error: true will ignore all failures in the comment step" feedback on #15</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes feedback on #15 #16

---

⚡ Quickly spin up Copilot coding agent tasks from anywhere on your macOS or Windows machine with Raycast.

[github-actions]

Warnings (4)

• plugins/dotnet-dnceng/skills/ci-analysis — ~2971 tokens (chars/4) — approaching 'comprehensive' range where gains diminish.
• plugins/dotnet-dnceng/skills/ci-crash-dump — ~3892 tokens (chars/4) — approaching 'comprehensive' range where gains diminish.
• plugins/dotnet-dnceng/skills/ci-crash-dump — no numbered workflow steps — agents follow sequenced procedures more reliably.
• plugins/dotnet-dnceng/skills/flow-analysis — ~5442 tokens (chars/4) — 'comprehensive' skills hurt performance. Consider splitting into 2-3 focused skills.

View workflow run

[Copilot]

Pull request overview

Updates the PR validation workflow so that failures in the “Post or update PR comment” step are only tolerated for fork-based PRs (where GITHUB_TOKEN is read-only and comment posting is expected to fail), while same-repo PRs will now fail the job if commenting errors occur—surfacing real problems like auth/rate-limit/API issues.

Changes:

• Make continue-on-error conditional on whether the pull request originates from a fork (github.event.pull_request.head.repo.fork).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.