Skip to content

Commit 7cf514b

Browse files
committed
Fix upgrade command. Closes #293
Closes #293
1 parent 9dd5d55 commit 7cf514b

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,13 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
66
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
77

88
> **Note**: odd version numbers, for example, `0.13.0`, are not included in this changelog. They are used to test the new features and fixes before the final release.
9+
910
## [0.26.3] - 2025-06-27
1011

12+
### Fixed:
13+
14+
- Commands: Fixed issue where brew update output would be shown in a notification when upgrading Dev Proxy via homebrew
15+
1116
## [0.26.2] - 2025-06-27
1217

1318
### Changed:

src/helpers.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,10 +122,11 @@ export const executeCommand = async (cmd: string, options: ExecOptions = {}): Pr
122122
return new Promise((resolve, reject) => {
123123
exec(cmd, options, (error, stdout, stderr) => {
124124
if (error) {
125-
reject(`exec error: ${error}`);
126-
} else if (stderr) {
127-
reject(`stderr: ${stderr}`);
125+
// Command failed with non-zero exit code
126+
reject(`exec error: ${error}${stderr ? `\nstderr: ${stderr}` : ''}`);
128127
} else {
128+
// Command succeeded (exit code 0), return stdout
129+
// Note: Many tools like brew send informational messages to stderr even on success
129130
resolve(stdout);
130131
}
131132
});

0 commit comments

Comments
 (0)