-
Notifications
You must be signed in to change notification settings - Fork 34
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
feat: enhance PR validation checks #37
Conversation
- Add JSON formatting and style validation - Add Python package (uvx) publication check - Add environment variable documentation validation - Add URL format validation - Add runtime validation - Update workflow for Python support Co-Authored-By: Michael Latman <[email protected]>
- Add better error messages and logging - Fix uvx package validation using pip install --dry-run - Improve JSON formatting validation - Add PR status emoji feedback - Fix pip configuration for uvx registry Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
Co-Authored-By: Michael Latman <[email protected]>
❌ PR validation failed. Please check the workflow logs for details. |
Co-Authored-By: Michael Latman <[email protected]>
❌ PR validation failed. Please check the workflow logs for details. |
Co-Authored-By: Michael Latman <[email protected]>
No new packages were added in this PR. |
✅ PR validation passed successfully! |
Co-Authored-By: Michael Latman <[email protected]>
❌ No new packages found or validation failed. |
✅ PR validation passed successfully! |
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
⚙️ Control Options:
Add "(aside)" to your comment to have me ignore it. |
This PR enhances the automated validation checks for MCP packages.
Changes
Add JSON formatting validation
Add Python package validation
Add environment variable validation
Enhance metadata validation
Testing
Link to Devin run: https://app.devin.ai/sessions/8f2357a02a9a404596469e6b4a2590a3