Skip to content

Add Skill-Validator & Evaluation workflow guidelines - #282

Merged
ViktorHofer merged 3 commits into
mainfrom
ViktorHofer-patch-1
Mar 8, 2026
Merged

Add Skill-Validator & Evaluation workflow guidelines#282
ViktorHofer merged 3 commits into
mainfrom
ViktorHofer-patch-1

Conversation

@ViktorHofer

Copy link
Copy Markdown
Member

Added guidelines for the Skill-Validator and evaluation workflow, emphasizing security measures for changes made from non-forked branches.

Added guidelines for the Skill-Validator and evaluation workflow, emphasizing security measures for changes made from non-forked branches.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds contributor-facing guidance on security constraints around modifying the skill-validator and evaluation workflows, clarifying how evaluation behaves for forked pull requests.

Changes:

  • Documented restrictions for changes to eng/skill-validator and .github/workflows/evaluation*.yml.
  • Noted that forked PRs do not apply modifications to these infrastructure paths during evaluation runs.

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

Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
ViktorHofer and others added 2 commits March 8, 2026 17:54
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ViktorHofer
ViktorHofer enabled auto-merge (squash) March 8, 2026 16:55
@ViktorHofer
ViktorHofer merged commit 1ae3b61 into main Mar 8, 2026
1 check passed
@ViktorHofer
ViktorHofer deleted the ViktorHofer-patch-1 branch March 8, 2026 18:39
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