Skip to content

Remove err/warn message from tooltip#100

Merged
brondani merged 3 commits into
Open-CMSIS-Pack:mainfrom
jthuangarm:clean-pr
Mar 24, 2026
Merged

Remove err/warn message from tooltip#100
brondani merged 3 commits into
Open-CMSIS-Pack:mainfrom
jthuangarm:clean-pr

Conversation

@jthuangarm
Copy link
Copy Markdown
Contributor

@jthuangarm jthuangarm commented Mar 24, 2026

Fixes

Changes

  • Removed the inclusion of error/warning messages from the tooltip in the StatusBar by eliminating the call to getSetupMessages and the display of its results in contextToolTip.

Screenshots

Before:
image

After:
image

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@jthuangarm jthuangarm marked this pull request as ready for review March 24, 2026 08:46
@jthuangarm jthuangarm requested a review from brondani March 24, 2026 08:46
Comment thread src/status-bar.ts Outdated
Copy link
Copy Markdown
Collaborator

@brondani brondani left a comment

Choose a reason for hiding this comment

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

LGTM

@brondani brondani merged commit 00d37cb into Open-CMSIS-Pack:main Mar 24, 2026
10 checks passed
@jthuangarm jthuangarm deleted the clean-pr branch March 24, 2026 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants