-
-
Notifications
You must be signed in to change notification settings - Fork 982
ci: add HOL skill-publish validate workflow #425
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
internet-dot
wants to merge
8
commits into
BeehiveInnovations:main
from
internet-dot:hol-skill-validate
Closed
Changes from all commits
Commits
Show all changes
8 commits
Select commit
Hold shift + click to select a range
45755bd
Add HOL skill-publish validate workflow
internet-dot 44eca2f
fix: pin actions to commit SHAs
internet-dot 2fab89c
fix: add SKILL.md and skill.json, pin actions to SHAs
8edb078
fix: update skill-publish action to latest version with validate mode
internet-dot 504d93a
fix: revert to stable skill-publish action, remove unsupported mode i…
3a48c75
fix: update skill-publish to latest with validate mode support
86e26c2
revert: pin skill-publish to stable v1 (new version has unbundled dep…
27081e9
fix: update skill-publish action with validate mode support
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,24 @@ | ||
| name: Validate Skill | ||
| on: | ||
| push: | ||
| branches: [main, master] | ||
| pull_request: | ||
| workflow_dispatch: | ||
|
|
||
| concurrency: | ||
| group: skill-validate-${{ github.ref }} | ||
| cancel-in-progress: true | ||
|
|
||
| jobs: | ||
| validate: | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 10 | ||
| permissions: | ||
| contents: read | ||
| steps: | ||
| - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4 | ||
| - name: Validate skill | ||
| uses: hashgraph-online/skill-publish@8f3c91d7d4cde7b104549e5f0cccdbb4cd7ee58d # v1 | ||
| with: | ||
| skill-dir: . | ||
| mode: validate | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| --- | ||
| name: pal-mcp-server | ||
| description: The power of Claude Code / GeminiCLI / CodexCLI + [Gemini / OpenAI / OpenRouter / Azure / Grok / Ollama / Custom Model / All Of The Above] working as one. | ||
| --- | ||
|
|
||
| # PAL MCP Server | ||
|
|
||
| The power of Claude Code / GeminiCLI / CodexCLI + [Gemini / OpenAI / OpenRouter / Azure / Grok / Ollama / Custom Model / All Of The Above] working as one. | ||
|
|
||
| ## Installation | ||
|
|
||
| See https://github.com/BeehiveInnovations/pal-mcp-server for setup instructions. | ||
|
|
||
| ## Available Tools | ||
|
|
||
| See the project README for the complete list of available MCP tools. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| { | ||
| "name": "pal-mcp-server", | ||
| "version": "0.1.0", | ||
| "description": "The power of Claude Code / GeminiCLI / CodexCLI + various AI models working as one.", | ||
| "author": { | ||
| "name": "BeehiveInnovations", | ||
| "url": "https://github.com/BeehiveInnovations" | ||
| }, | ||
| "homepage": "https://github.com/BeehiveInnovations/pal-mcp-server", | ||
| "repository": "https://github.com/BeehiveInnovations/pal-mcp-server", | ||
| "keywords": ["mcp", "codex"], | ||
| "license": "MIT" | ||
| } |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The workflow validates
skill-dir: ., but this repository does not contain the requiredSKILL.mdandskill.jsonfiles at the root (or elsewhere); I checked the tree withrg --files | rg 'SKILL\.md|skill\.json'and found none.hashgraph-online/skill-publishdocumentsskill-diras 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 👍 / 👎.