Skip to content

Skip code workflows for changes limited to agent skills - #141

Merged
erseco merged 1 commit into
mainfrom
feature/skip-code-workflows-for-skills
Sep 22, 2026
Merged

erseco merged 1 commit into
mainfrom
feature/skip-code-workflows-for-skills

Conversation

@erseco

@erseco erseco commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Changes limited to .agents/skills/** and .claude/skills/** no longer start the Moodle/PHP/database test matrix or Playground preview workflow. Code changes mixed with skill updates still run normally.

Keep branch selection, manual runs, scheduled skill updates and release automation unchanged. Remove the setup-php reference to the undefined matrix.extensions property, which already evaluated to an empty value and was rejected by actionlint.

Validation: actionlint passes on both workflows; parsed YAML checks cover skills-only, mixed, code-only, workflow-change and non-skill guidance cases; git diff --check passes. Rulesets and classic branch protection have no required status checks on main.


Moodle Playground Preview

The changes in this pull request can be previewed and tested using a Moodle Playground instance.

Preview in Moodle Playground

ℹ️ The eXeLearning editor is fetched from the shared release and unpacked into the plugin when the playground boots, so the first load may take a few extra seconds. ELPX upload, viewer and preview work normally.

@codecov-commenter

codecov-commenter commented Sep 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.07%. Comparing base (97a06ef) to head (01af865).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #141   +/-   ##
=========================================
  Coverage     94.07%   94.07%           
  Complexity      793      793           
=========================================
  Files            47       47           
  Lines          3563     3563           
=========================================
  Hits           3352     3352           
  Misses          211      211           
Flag Coverage Δ
javascript 96.00% <ø> (ø)
php 93.99% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
PHP (server-side) 93.99% <ø> (ø)
JavaScript (SCORM tracker) 96.00% <ø> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@erseco
erseco merged commit e6ffdfb into main Sep 22, 2026
25 checks passed
@erseco
erseco deleted the feature/skip-code-workflows-for-skills branch September 22, 2026 12:56
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.

2 participants