Merged
Conversation
This adds two Vite projects (a JavaScript build tool) for the lobby and game clients, which need to be built as separate bundles. These are just skeletons for now, files will be moved and altered in following commits in an attempt to minimise git conflicts for anyone who might have .js changes in a branch or fork.
This moves all the static .js from webtiles into the appropriate project src file. This commit literally just moves the files so that git can track the history of the files and (hopefully) help resolve conflicts if any other developers have touched these files.
Get the new bundling system working, and use the biome tool to apply linting fixes and auto formatting to .js files across the board.
Add necessary steps in the Makefile and in CI to produce the .js bundles as required when dependencies have changed, running `npm ci` as necessary if any package.json or related files have changed. CI pipelines have a nodejs installation added. I have not included nodejs in deps.py, as the installation varies from system to system. This must be documented in a further commit (once I am confident what the documentation needs to be exactly).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.