Skip to content

Fix change detection on server config files#2680

Open
Measurity wants to merge 1 commit intoSubnauticaNitrox:masterfrom
Measurity:fix-config-change-detect
Open

Fix change detection on server config files#2680
Measurity wants to merge 1 commit intoSubnauticaNitrox:masterfrom
Measurity:fix-config-change-detect

Conversation

@Measurity
Copy link
Collaborator

FileConfigurationSource first argument, Path, should be relative to the directory given to IFileProvider.

With this fix, change detect should work.

Needs testing on Windows.

Testing

  1. Start server
  2. Change server.Development.json by setting PortForward to true.
  3. Notice: after a few seconds that the server will port forward the default port.
  4. Change ServerPort in server.cfg to 11001 (is in save folder of server)
  5. Notice: after a few seconds that the port forward be will removed from 11000 and set to 11001.

@Measurity Measurity added the Area: setup Related to installing, configuring or how compatible Nitrox is with other tools label Mar 12, 2026
@github-actions
Copy link

github-actions bot commented Mar 12, 2026

Test Results

239 tests  ±0   238 ✅ ±0   9s ⏱️ -2s
  1 suites ±0     1 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 8a2ceee. ± Comparison against base commit 911c103.

♻️ This comment has been updated with latest results.

@Measurity Measurity force-pushed the fix-config-change-detect branch from 651ade7 to 8a2ceee Compare March 12, 2026 23:39
@dartasen dartasen added this to the 1.9 milestone Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: setup Related to installing, configuring or how compatible Nitrox is with other tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants