-
-
Notifications
You must be signed in to change notification settings - Fork 522
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
publisher-github: Asset name with spaces doesn't match already uploaded asset name #1759
Comments
Ideally something like octokit would have a way for us to sanitize the asset filenames so we can do this check better. |
@malept Github documentation suggests to contact support https://github.com/contact
|
Preflight Checklist
Issue Details
Expected Behavior
publisher-github
correctly detect already published asset.Actual Behavior
publisher-github
doesn't match already uploaded asset name.To Reproduce
My App.dmg
Additional Information
The following check is not valid because:
GitHub renames asset filenames that have special characters, non-alphanumeric characters, and leading or trailing periods.
https://github.com/electron-userland/electron-forge/blob/a2de8ea5ec7d66725cd20ef8dabbd727843b52a3/packages/publisher/github/src/PublisherGithub.ts#L97-L99
The text was updated successfully, but these errors were encountered: