Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MIRROR] Juke Build 0.8.1 #230

Merged
merged 2 commits into from Aug 15, 2021
Merged

[MIRROR] Juke Build 0.8.1 #230

merged 2 commits into from Aug 15, 2021

Conversation

ms-mirror-bot
Copy link
Collaborator

Original PR: tgstation/tgstation#60404

About The Pull Request

With this PR, I have decided to move some build, test and lint related tasks to build.js.

Here is a full list of targets/parameters, to get a taste of what you can run with this thing:

$ tools/build/build --help
:: Juke Build version 0.8.1
:: Available targets:
 - all
 - build
 - clean
 - dist-clean
 - dm-maps-include
 - dm
 - dm-test
 - lint
 - server
 - test
 - tg-font
 - tgs
 - tgui-analyze
 - tgui-dev
 - tgui-eslint
 - tgui-lint
 - tgui
 - tgui-test
 - tgui-tsc
 - yarn
:: Available parameters:
 --ci (type: boolean)
 --define, -D (type: string[])
 --port, -p (type: string)
  • server compiles and hosts the server in one go.
    • tools/build/build server --port 10050
  • test runs DM unit tests.
  • all runs all of the above in the most efficient manner.

Updated the CI pipeline to reflect this change, removed the python script to generate a list of map includes (because build.js now does it automatically based on provided defines). Added restoring of Yarn cache in CI, which saves about a minute.

A lot of small improvements to Juke, detailed changelog is here.

Changed .mdme extension to .m.dme because it results in a more organized set of files, i.e. .m.dme, .m.dmb, .m.rsc.

Bumped node version because earlier versions do not support ES modules without extra experimental flags.

Added THREE new bat files for dummies:

  • CLEAN.bat (for fixing random issues)
  • RUN_TESTS.bat for an offline CI run.
  • RUN_SERVER.bat for running the server, very simple (default port is 1337).

@Koshenko Koshenko merged commit bd7ba40 into master Aug 15, 2021
@Koshenko Koshenko deleted the upstream-merge-60404 branch August 21, 2021 22:31
@Koshenko Koshenko added the Mirror PR mirrored from upstream label Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mirror PR mirrored from upstream
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants