-
-
Notifications
You must be signed in to change notification settings - Fork 523
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(publisher-bitbucket): initial publish publisher-bitbucket #571
Conversation
…er-bitbucket ISSUES CLOSED: #487
// TODO: Consider checking if the files already exist at the current version and abort if so? | ||
|
||
await asyncOra(`Uploading result (${index + 1}/${makeResults.length})`, async () => { | ||
// TODO: See if we can use this same API for bitbucket server, we could take in a `host` config if so |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
"version": "6.0.0-beta.28", | ||
"description": "Github publisher for Electron Forge", | ||
"repository": "https://github.com/electron-userland/electron-forge", | ||
"author": "Samuel Attard", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can I chuck my name here? XD
0638f7f
to
625eb95
Compare
Ugh. Think I just need to delete the test command |
const encodedUserAndPass = Buffer.from(`${auth.username}:${auth.appPassword}`).toString('base64'); | ||
|
||
if (!(hasRepositoryConfig && config.repository.owner && config.repository.name)) { | ||
throw 'In order to publish to Bitbucket you must set the "repository.owner" and "repository.name" properties in your forge config. See the docs for more info'; // eslint-disable-line |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I generally prefer to specify which eslint rule I'm ignoring here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I honestly couldn't tell you which it was, I copied it from the GitHub one, probably max-len.
I'll update both tomorrow.
/** | ||
* If true, will override an existing version of a published release (will throw an error otherwise). | ||
*/ | ||
overrideExistingVersion?: boolean; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bitbucket downloads don't technically have versions so this could be confusing to people, how about overrideExistingFiles?: boolean
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup. Sure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went with replaceExistingFiles
in the end if that's cool.
d5e6fb1
to
73653e0
Compare
…ublisher-bitbucket
73653e0
to
858a7e2
Compare
Fixes #487