ci: add HOL skill-publish validate workflow#425
ci: add HOL skill-publish validate workflow#425internet-dot wants to merge 8 commits intoBeehiveInnovations:mainfrom
Conversation
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 45755bdb9d
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| uses: hashgraph-online/skill-publish@v1 | ||
| with: | ||
| mode: validate | ||
| skill-dir: . |
There was a problem hiding this comment.
Point skill-dir to an actual skill package
The workflow validates skill-dir: ., but this repository does not contain the required SKILL.md and skill.json files at the root (or elsewhere); I checked the tree with rg --files | rg 'SKILL\.md|skill\.json' and found none. hashgraph-online/skill-publish documents skill-dir as required and expects those files, so this step will fail for every push/PR and can block CI until the path is changed to a real skill package directory (or the workflow is scoped to repos/paths that have one).
Useful? React with 👍 / 👎.
This repo already has enough skill metadata to validate in CI.
A couple of repo-specific details I checked first:
This adds one workflow for
.and leaves the runtime code alone.It only runs the schema and trust checks for the skill metadata in validate mode.
If you would rather place the workflow under a different filename or point it at a different skill directory, I can adjust the branch.