Skip to content

fix(bundler): modify version in output file name when bundling wix#8182

Merged
lucasfernog merged 2 commits intotauri-apps:devfrom
moka-ayumu:fix/wix-output-name-version
Nov 7, 2023
Merged

fix(bundler): modify version in output file name when bundling wix#8182
lucasfernog merged 2 commits intotauri-apps:devfrom
moka-ayumu:fix/wix-output-name-version

Conversation

@moka-ayumu
Copy link
Copy Markdown
Contributor

@moka-ayumu moka-ayumu commented Nov 6, 2023

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Docs
  • New Binding issue #___
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change?

  • Yes, and the changes were approved in issue #___
  • No

Checklist

  • When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
  • A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
  • I have added a convincing reason for adding this feature, if necessary

Other information

When wix bundling a project whose current version is 0.0.0-0, the file name appears as 0.0.0.0. This is not identical to other files. In addition, GitHub Action searches for 0.0.0-0 files, so only related msi files cannot be uploaded together.

@moka-ayumu moka-ayumu requested a review from a team as a code owner November 6, 2023 13:43
@lucasfernog
Copy link
Copy Markdown
Member

Thank you for your contribution! We actually enforce signed commits, but I'll force merge this one to save time. Would be cool if you could set it up for next time :)

@lucasfernog lucasfernog merged commit 34196e2 into tauri-apps:dev Nov 7, 2023
@moka-ayumu
Copy link
Copy Markdown
Contributor Author

Thank you for your contribution! We actually enforce signed commits, but I'll force merge this one to save time. Would be cool if you could set it up for next time :)

Thanks, I'm just setting it up now! I'll sign the commit next time 👍

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

Successfully merging this pull request may close these issues.

2 participants