Skip to content

Conversation

wjt
Copy link
Member

@wjt wjt commented Oct 25, 2024

Like endlessm/moddable-pong#5 and endlessm/moddable-platformer#4.

Plus a change to set the version in the project file when exporting (again like our other repos).

https://phabricator.endlessm.com/T35699

wjt added 4 commits October 25, 2024 14:59
This preset is just the default settings for the Web target (as of v4.3.stable.flathub [77dcf97d8], at least). The important line is: variant/thread_support=false This selects a build of the Godot web runtime that does not use threads; and hence does not use SharedArrayBuffer; and hence can be used on GitHub Pages where the necessary headers to allow SharedArrayBuffer to be used cannot be set. https://phabricator.endlessm.com/T35699
While it is not necessarily useful to have these demos in their default state published to the web, this workflow can be used by a learner who has customised the demos in their own repo. The workflow is designed to fail gracefully if GitHub Pages is not available (which is typically the case in a private repo owned by a free account) or not configured correctly. It also only runs on releases or when manually triggered, to avoid running it each time a learner pushes or merges to main. https://phabricator.endlessm.com/T35699
This workflow is intended to be useful to learners who have forked these demos as a basis for their own modified games. But rather than using an actual fork, they will likely have downloaded the project from the asset library and committed it to a fresh repository. https://phabricator.endlessm.com/T35699
@cassidyjames
Copy link
Member

cassidyjames commented Oct 25, 2024

Reminder for someone in @endlessm/endless-developers with admin access to the repo to set the repo URL to https://endlessm.github.io/godot-block-coding-demos/ after merging so it's more discoverable!

Edit: oh, and please enable GitHub Pages in the repo settings if it's not already.

Edit ×2: in the wise words of @wjt in endlessm/moddable-platformer#4 (comment):

Maybe the repo URL would be better as the assetlib entry. The web version of the game is not really the interesting part - the point is to get the game into Godot and hack it. We could adjust the readme and mention the web build there.

Copy link
Member

@cassidyjames cassidyjames left a comment

Choose a reason for hiding this comment

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

Let's go!

@cassidyjames cassidyjames merged commit ec93856 into main Oct 25, 2024
@cassidyjames cassidyjames deleted the T35699-publish-to-github-pages branch October 25, 2024 19:01
@manuq
Copy link
Contributor

manuq commented Oct 25, 2024

Excellent! Should we put the link to the game online in the README?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants