Skip to content

feat(newserver): add Hytale server support#4928

Draft
andrewqsantos wants to merge 2 commits into
GameServerManagers:developfrom
andrewqsantos:feature/hytale-server
Draft

feat(newserver): add Hytale server support#4928
andrewqsantos wants to merge 2 commits into
GameServerManagers:developfrom
andrewqsantos:feature/hytale-server

Conversation

@andrewqsantos
Copy link
Copy Markdown

Description

Adds LinuxGSM support for Hytale server.

Fixes #4867

Type of change

  • Bug fix (a change which fixes an issue).
  • New feature (a change which adds functionality).
  • New Server (new server added).
  • Refactor (restructures existing code).
  • Comment update (typo, spelling, explanation, examples, etc).

Testing

Please list the exact validation you performed and the outcome.

  • Commands/tests run:
    • git diff --check upstream/develop...HEAD
    • datadir=lgsm/data bash .github/scripts/serverlist-validate.sh
    • datadir=lgsm/data bash .github/scripts/serverlist-validate-game-icons.sh
    • bash -n on changed shell modules
    • bash tests/hytale-update-harness.sh
    • prettier --check --ignore-unknown on changed shell files
  • Result: Passed
  • Environment used (distro/version): macOS local workspace, shell validation

Risk and rollback

  • Risk level: medium
  • Rollback plan: Revert this PR to remove Hytale support and restore previous update/monitor behavior.

Breaking changes

  • No breaking changes.
  • Breaking changes included (describe below).

Documentation impact

  • No documentation update required.
  • User documentation update required.
  • Developer documentation update required.

Checklist

PR will not be merged until all steps are complete.

  • This pull request links to an issue.
  • This pull request uses the develop branch as its base.
  • This pull request subject follows the Conventional Commits standard.
  • This code follows the style guidelines of this project.
  • I have performed a self-review of my code.
  • I have provided a detailed enough description of this PR.

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.

1 participant