Skip to content

fix: outdated error message instructions #196

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 30, 2025
Merged

fix: outdated error message instructions #196

merged 1 commit into from
Jun 30, 2025

Conversation

@iloveitaly
Copy link
Contributor

@iloveitaly iloveitaly commented Jun 30, 2025 •
edited by qodo-merge-pro bot
Loading

PR Type

Bug fix

Description

  • Updated error message instructions to use correct command syntax

  • Changed 'mcpm client set' to 'mcpm target set @' in error messages

Changes diagram

flowchart LR
  A["Error Messages"] --> B["Updated Command Syntax"]
  B --> C["mcpm target set @<client-name>"]
Loading

Changes walkthrough 📝

Relevant files
Bug fix
pop.py
Fix client switching error message                                             

src/mcpm/commands/target_operations/pop.py

  • Updated error message to use correct command syntax 'mcpm target set
    @'
    		• +1/-1     
    display.py
    Fix standardized client error message                                       

    src/mcpm/utils/display.py

  • Updated standardized client error message with correct command syntax
    		• +1/-1     
    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
    •

    Copilot AI review requested due to automatic review settings June 30, 2025 20:17
    @qodo-merge-pro
    Copy link
    Contributor

    qodo-merge-pro bot commented Jun 30, 2025

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @qodo-merge-pro
    Copy link
    Contributor

    qodo-merge-pro bot commented Jun 30, 2025

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    Copilot AI reviewed Jun 30, 2025
    View reviewed changes
    Copy link
    Contributor

    Copilot AI left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

    @niechen niechen merged commit 49c9be1 into pathintegral-institute:main Jun 30, 2025
    3 of 4 checks passed
    mcpm-semantic-release bot pushed a commit that referenced this pull request Jun 30, 2025
    @semantic-release-bot
    chore(release): 1.14.1 [skip ci]
    21da6a8 
    ## [1.14.1](v1.14.0...v1.14.1) (2025-06-30)

### Bug Fixes

* outdated error message instructions ([#196](#196)) ([49c9be1](49c9be1))
    @mcpm-semantic-release
    Copy link

    mcpm-semantic-release bot commented Jun 30, 2025

    🎉 This PR is included in version 1.14.1 🎉

    The release is available on GitHub release

    Your semantic-release bot 📦🚀

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Reviewers

    Copilot code review Copilot Copilot left review comments

    @niechen niechen niechen approved these changes

    Assignees

    No one assigned

    Labels

    Review effort 1/5

    Projects

    None yet

    Milestone

    No milestone

    Development

    Successfully merging this pull request may close these issues.

    2 participants

    @iloveitaly @niechen