Skip to content

Commit 6228318

Browse files
docs: remove redundant validation steps from GitHub Actions guide
Remove 'Test Before Publishing' and 'Notification Setup' sections from github-actions.md as these steps are redundant since the publisher API already handles validation. Fixes #432 Co-authored-by: adam jones <domdomegg@users.noreply.github.com>
1 parent 773f094 commit 6228318

File tree

1 file changed

+0
-35
lines changed

1 file changed

+0
-35
lines changed

docs/guides/publishing/github-actions.md

Lines changed: 0 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -272,41 +272,6 @@ Only publish to registry after package publishing succeeds:
272272
run: ./mcp-publisher publish
273273
```
274274

275-
### 3. Test Before Publishing
276-
277-
Include validation in your workflow:
278-
279-
```yaml
280-
- name: Validate server.json
281-
run: |
282-
# Validate against schema
283-
curl -s https://static.modelcontextprotocol.io/schemas/2025-07-09/server.schema.json | \
284-
jq . > schema.json
285-
npx ajv-cli validate -s schema.json -d server.json
286-
287-
# Test with publisher
288-
./mcp-publisher init --validate-only
289-
```
290-
291-
### 4. Notification Setup
292-
293-
Get notified of publishing status:
294-
295-
```yaml
296-
- name: Notify on success
297-
if: success()
298-
run: |
299-
curl -X POST "${{ secrets.WEBHOOK_URL }}" \
300-
-H "Content-Type: application/json" \
301-
-d '{"text": "✅ MCP server ${{ github.repository }} published successfully"}'
302-
303-
- name: Notify on failure
304-
if: failure()
305-
run: |
306-
curl -X POST "${{ secrets.WEBHOOK_URL }}" \
307-
-H "Content-Type: application/json" \
308-
-d '{"text": "❌ MCP server ${{ github.repository }} publishing failed"}'
309-
```
310275

311276
## Troubleshooting
312277

0 commit comments

Comments
 (0)