Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit 1e0bbe7

Browse files
committed
Merge pull request #1784 from adobe/glenn/update-notification
Don't show update notification when running from GitHub source
2 parents 82ef833 + 9ab99e4 commit 1e0bbe7

File tree

1 file changed

+10
-1
lines changed

1 file changed

+10
-1
lines changed

src/utils/UpdateNotification.js

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ define(function (require, exports, module) {
3939
Global = require("utils/Global");
4040

4141
// Extract current build number from package.json version field 0.0.0-0
42-
var _buildNumber = /-([0-9]+)/.exec(brackets.metadata.version)[1];
42+
var _buildNumber = Number(/-([0-9]+)/.exec(brackets.metadata.version)[1]);
4343

4444
// PreferenceStorage
4545
var _prefs = PreferencesManager.getPreferenceStorage(module.id, {lastNotifiedBuildNumber: 0});
@@ -283,6 +283,15 @@ define(function (require, exports, module) {
283283
// Get all available updates
284284
var allUpdates = _stripOldVersionInfo(versionInfo, _buildNumber);
285285

286+
// When running directly from GitHub source (as opposed to
287+
// an installed build), _buildNumber is 0. In this case, if the
288+
// test is not forced, don't show the update notification icon or
289+
// dialog.
290+
if (_buildNumber === 0 && !force) {
291+
result.resolve();
292+
return;
293+
}
294+
286295
if (allUpdates) {
287296
// Always show the "update available" icon if any updates are available
288297
var $updateNotification = $("#update-notification");

0 commit comments

Comments
 (0)