Skip to content

Fix/351 asset store duplicate - #352

Merged
NathanLovato merged 4 commits into
GDQuest:mainfrom
DevTwilight:fix/351-asset-store-duplicate
Sep 14, 2026
Merged

NathanLovato merged 4 commits into
GDQuest:mainfrom
DevTwilight:fix/351-asset-store-duplicate

Conversation

@DevTwilight

Copy link
Copy Markdown
Contributor

Please check if the PR fulfills these requirements:

  • The commit message follows our guidelines.
  • For bug fixes and features:
    • You tested the changes.

Related issue (if applicable): #351

What kind of change does this PR introduce?

This PR fixes the release workflow so that the two Godot add-ons are packaged into separate ZIP archives instead of being bundled together.

Does this PR introduce a breaking change?

No.

New feature or change

What is the current behavior?

The release workflow recursively packages the entire addons/ directory into a single godot-addon.zip archive. This causes both GDQuest_GDScript_formatter and GDQuest_GDScript_formatter_standalone to be included in the same archive.

What is the new behavior?

The release workflow creates two separate archives:

  • godscript-formatter.zip containing GDQuest_GDScript_formatter
  • godscript-formatter-standalone.zip containing GDQuest_GDScript_formatter_standalone

Both archives are uploaded as release assets.

Other information

This addresses issue #351 by allowing each Godot add-on to be distributed independently without packaging both add-ons together.

Comment thread .github/workflows/release.yml Outdated
@NathanLovato

Copy link
Copy Markdown
Contributor

Thank you. I just changed the file names to be stable because some links across the web depend on them.
I'll also merge the WinGet workflow. That will force me to finally get the token up!

@NathanLovato
NathanLovato merged commit 46c43b0 into GDQuest:main Sep 14, 2026
1 check passed
@DevTwilight
DevTwilight deleted the fix/351-asset-store-duplicate branch September 14, 2026 08:50
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