-
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
fix: implement update command and improve error handling #55
Conversation
- Added update command to CLI - Connected updatePackage() from auto-update.ts - Improved error handling for update process - Added proper version comparison messaging Co-Authored-By: Michael Latman <[email protected]>
🤖 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. |
❌ No new packages found or validation failed. |
✅ PR validation passed successfully! |
- Add silent mode to updatePackage - Automatically check for updates before running commands - Maintain graceful failure handling Co-Authored-By: Michael Latman <[email protected]>
❌ No new packages found or validation failed. |
✅ PR validation passed successfully! |
Co-Authored-By: Michael Latman <[email protected]>
❌ No new packages found or validation failed. |
✅ PR validation passed successfully! |
- Introduced a new "test:update" script in package.json for testing updates. - Updated the updatePackage function to use 'npm install -g' instead of 'npx' for installing updates. - Improved error handling to provide detailed output for both stdout and stderr during the update process. - Adjusted unit tests to reflect changes in the update command and error handling. Co-Authored-By: Michael Latman <[email protected]>
❌ No new packages found or validation failed. |
✅ PR validation passed successfully! |
Link to Devin run: https://app.devin.ai/sessions/928989f6b55947ea8be32ff3c8a57a6e