Skip to content

Conversation

@wjt wjt force-pushed the push-qtozlqwsozvz branch from f6c9f3d to 0d0c9d8 Compare September 24, 2024 10:37
@wjt wjt force-pushed the push-qtozlqwsozvz branch from 0d0c9d8 to 76386f1 Compare September 24, 2024 10:40
@wjt wjt requested a review from manuq September 24, 2024 10:40
Comment on lines +4 to +6
# Exclude GitHub Actions-related files from published asset
/.github export-ignore
/asset-template.json.hb export-ignore
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should also exclude /asset-sources/ to reduce the download size.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pushed an extra patch to do that.

This will roughly halve the size of the download when people fetch this from the Godot Asset Library.
@dbnicholson dbnicholson marked this pull request as ready for review September 24, 2024 18:07
@dbnicholson
Copy link
Member

Marked as ready now that the actions secrets are in place.

Copy link
Member

@dbnicholson dbnicholson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks good to me.

@dbnicholson dbnicholson merged commit cca64a3 into main Sep 24, 2024
@dbnicholson dbnicholson deleted the push-qtozlqwsozvz branch September 24, 2024 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants