Skip to content

Commit ef3899e

Browse files
authored
Merge pull request electron#268 from electron/check-heroku-login-status-before-release
check Heroku login status before release
2 parents a9ae2a7 + 9436ebc commit ef3899e

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

script/release.js

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ const token = process.env.ELECTRON_API_DEMO_GITHUB_TOKEN
1010
const version = require('../package').version
1111

1212
checkToken()
13+
.then(checkHerokuLoginStatus)
1314
.then(zipAssets)
1415
.then(createRelease)
1516
.then(uploadAssets)
@@ -28,6 +29,20 @@ function checkToken () {
2829
}
2930
}
3031

32+
function checkHerokuLoginStatus () {
33+
return new Promise((resolve, reject) => {
34+
console.log('Checking Heroku login status')
35+
36+
childProcess.exec('heroku whoami', (error, stdout, stderr) => {
37+
if (error) {
38+
reject('You are not logged in to GitHub\'s Heroku Enterprise account. To log in, run this command:\n$ heroku login --sso')
39+
} else {
40+
resolve()
41+
}
42+
})
43+
})
44+
}
45+
3146
function zipAsset (asset) {
3247
return new Promise((resolve, reject) => {
3348
const assetBase = path.basename(asset.path)

0 commit comments

Comments
 (0)