Skip to content

Adding a 'watch' mode to most projects#420

Merged
mcottontensor merged 26 commits intoEpicGamesExt:masterfrom
mcottontensor:build-cleanup
Jan 24, 2025
Merged

Adding a 'watch' mode to most projects#420
mcottontensor merged 26 commits intoEpicGamesExt:masterfrom
mcottontensor:build-cleanup

Conversation

@mcottontensor
Copy link
Copy Markdown
Collaborator

@mcottontensor mcottontensor commented Jan 22, 2025

Relevant components:

  • Signalling server
  • Common library
  • Frontend library
  • Frontend UI library
  • Matchmaker
  • Platform scripts
  • SFU

Problem statement:

#332
The build process is finicky and error prone with knowing what to build and when. Some build scripts are even broken in some situations. We want to automate or semi-automate the build process so that development is more streamlined.

Solution

After much fussing about with mono repo tools and webpack etc. I found that using a series of nodemon scripts along with tsc watch mode and webpack was the simplest solution.
There is now a single script called develop in the signalling server project that will go though all the components and kick off a watch process that will not only watch for source changes but also dependency changes and auto build and when needed restart the signalling server.
This should allow you to start the develop script and then make changes to whatever component you need, including frontend changes, and have them working and served immediately.

Documentation

Updated SS launch docs mentioning npm run develop

Test Plan and Compatibility

See above.

Copy link
Copy Markdown
Contributor

@lukehb lukehb left a comment

Choose a reason for hiding this comment

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

Some comments about what I believe to be load bearing configuration of how ESM modules were being built.

Additionally, we should land docs changes for how to use watch in this PR too.

Besides that, PR and approach looks good!

@mcottontensor mcottontensor merged commit 5f8418f into EpicGamesExt:master Jan 24, 2025
@mcottontensor mcottontensor deleted the build-cleanup branch January 24, 2025 00:31
@mcottontensor
Copy link
Copy Markdown
Collaborator Author

💚 All backports created successfully

Status Branch Result
UE5.5

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

mcottontensor added a commit that referenced this pull request Feb 9, 2025
[UE5.5] Merge pull request #420 from mcottontensor/build-cleanup
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