Skip to content

Do not show MsgBox if /SUPPRESSMSGBOXES is active - #1654

Merged
raivisdejus merged 2 commits into
chidiwilliams:mainfrom
sepo:main
Oct 16, 2026
Merged

raivisdejus merged 2 commits into
chidiwilliams:mainfrom
sepo:main

Conversation

@sepo

@sepo sepo commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Hey, I recommend suppressing the MsgBox when /SUPPRESSMSGBOXES is active. Otherwise, an unattended or silent uninstallation is not possible because the MsgBox will appear regardless.

@raivisdejus

Copy link
Copy Markdown
Collaborator

@sepo Thansk for your contribution!

@raivisdejus
raivisdejus enabled auto-merge (squash) October 16, 2026 18:12
@codecov

codecov Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.54%. Comparing base (d05a48a) to head (6fc1a85).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1654   +/-   ##
=======================================
  Coverage   83.54%   83.54%           
=======================================
  Files         116      116           
  Lines       13851    13851           
=======================================
  Hits        11572    11572           
  Misses       2279     2279           
Flag Coverage Δ
Linux 82.80% <ø> (ø)
macOS 83.28% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@raivisdejus
raivisdejus merged commit c02e63f into chidiwilliams:main Oct 16, 2026
15 of 16 checks passed
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