Skip to content
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

Don't set GitHub Releases draft title #3683

Closed
sindresorhus opened this issue Feb 12, 2019 · 30 comments
Closed

Don't set GitHub Releases draft title #3683

sindresorhus opened this issue Feb 12, 2019 · 30 comments

Comments

@sindresorhus
Copy link

  • Version: 20.38.5
  • Target: macOS

When using auto-publishing to GitHub Releases, it creates a new release draft and sets the title to the version number. This is moot as GitHub will fall back to the tag name when no title is set, which is the version number.

@stale
Copy link

stale bot commented Jul 1, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the backlog label Jul 1, 2019
@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Jul 2, 2019
@stale
Copy link

stale bot commented Aug 31, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the backlog label Aug 31, 2019
@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Aug 31, 2019
@stale
Copy link

stale bot commented Oct 30, 2019

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the backlog label Oct 30, 2019
@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Oct 30, 2019
@stale
Copy link

stale bot commented Dec 29, 2019

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the backlog label Dec 29, 2019
@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Dec 29, 2019
@stale
Copy link

stale bot commented Feb 27, 2020

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the backlog label Feb 27, 2020
@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Feb 27, 2020
@stale
Copy link

stale bot commented Apr 27, 2020

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the backlog label Apr 27, 2020
@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Apr 27, 2020
@stale
Copy link

stale bot commented Jun 26, 2020

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the backlog label Jun 26, 2020
@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Jun 26, 2020
@sindresorhus
Copy link
Author

@stalebot We've been seeing each other for a while now. I think it's time you met my parents.

@LuKks
Copy link

LuKks commented Jun 27, 2020

We just need to remove line 210, right? name is optional: /v3/repos/releases/#create-a-release

private createRelease() {
return this.githubRequest<Release>(`/repos/${this.info.owner}/${this.info.repo}/releases`, this.token, {
tag_name: this.tag,
name: this.version,
draft: this.releaseType === "draft",
prerelease: this.releaseType === "prerelease",
})
}

createRelease is used here:

if (this.options.publish === "always" || getCiTag() != null) {
log.info({
reason: "release doesn't exist",
...logFields,
}, `creating GitHub release`)
return this.createRelease()
}

Assuming that this.options.publish in always is the auto-publish option.

@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Aug 27, 2020
@stale
Copy link

stale bot commented Oct 28, 2020

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the backlog label Oct 28, 2020
@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Oct 28, 2020
@stale
Copy link

stale bot commented Dec 28, 2020

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the backlog label Dec 28, 2020
@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Dec 28, 2020
@stale
Copy link

stale bot commented Mar 20, 2021

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the backlog label Mar 20, 2021
@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Mar 20, 2021
@mmaietta
Copy link
Collaborator

Hi @sindresorhus , please give this a shot. You can use patch-package to always have this patch applied as a postinstall hook.
Filename electron-publish+22.10.5.patch

diff --git a/node_modules/electron-publish/out/gitHubPublisher.js b/node_modules/electron-publish/out/gitHubPublisher.js
index 75e5ca2..e24f0f7 100644
--- a/node_modules/electron-publish/out/gitHubPublisher.js
+++ b/node_modules/electron-publish/out/gitHubPublisher.js
@@ -273,7 +273,7 @@ class GitHubPublisher extends _publisher().HttpPublisher {
   createRelease() {
     return this.githubRequest(`/repos/${this.info.owner}/${this.info.repo}/releases`, this.token, {
       tag_name: this.tag,
-      name: this.version,
+      // name: this.version,
       draft: this.releaseType === "draft",
       prerelease: this.releaseType === "prerelease"
     });

That should deactivate the draft title per the earlier comment.

@sindresorhus
Copy link
Author

Thanks, but it's not important enough to deal with patch-package. I just manually remove the title until this is properly fixed electron-builder.

@stale
Copy link

stale bot commented Jun 4, 2021

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the backlog label Jun 4, 2021
@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Jun 4, 2021
@stale
Copy link

stale bot commented Sep 19, 2021

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the backlog label Sep 19, 2021
@sindresorhus
Copy link
Author

Please keep this open.

@stale stale bot removed the backlog label Sep 19, 2021
mmaietta pushed a commit to mmaietta/electron-builder that referenced this issue Sep 21, 2021
mmaietta pushed a commit to mmaietta/electron-builder that referenced this issue Dec 29, 2021
mmaietta pushed a commit that referenced this issue Jan 10, 2022
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

No branches or pull requests

3 participants