Feat fedora support#167
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds Linux distro-family detection to install SteamCMD dependencies on both Debian-based and RHEL-based systems (addressing issue #166), and also introduces a UI/API flow for updating individual Steam Workshop mods from the SLP mod list.
Changes:
- Add Debian vs RHEL distro detection (via
/etc/os-release) and install SteamCMD deps usingapt-getordnf. - Add “Update” button per mod in the SLP UI, backed by a new
/api/v2/steamcmd/updatemodendpoint. - Update dev tooling (devcontainer) and CLI dev command naming/behavior.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
src/steamcmd/steamcmd-helper.go |
Adds distro-family detection and separate dependency installers for Debian and RHEL. |
src/web/slp-launchpad.go |
Adds an API handler to update a single workshop mod. |
src/web/routes.go |
Registers the new single-mod update route. |
UIMod/onboard_bundled/assets/js/slp.js |
Adds per-mod update action and adjusts button loading state behavior. |
UIMod/onboard_bundled/assets/css/config.css |
Styles the new per-mod update button. |
src/cli/devcommands.go |
Reworks a dev command to accept arguments for workshop downloads. |
src/cli/commands.go |
Registers the renamed workshop download CLI command. |
src/config/config.go |
Bumps app version to 5.13.3. |
.devcontainer/Dockerfile |
Tweaks Node.js install step (removes global npm@latest update). |
.devcontainer/devcontainer.json |
Changes devcontainer extensions and post-create command. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| _, err := steamcmd.DownloadWorkshopItems(workshopHandles) | ||
| if err != nil { | ||
| logger.Core.Error("Error downloading workshop items: " + err.Error()) | ||
| } | ||
| return nil |
There was a problem hiding this comment.
@JacksonTheMaster leaving this up to you
| Version = "5.13.3" | ||
| Branch = "release" |
There was a problem hiding this comment.
@JacksonTheMaster leaving versioning stuff up to you, not sure how you do it
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Cannot merge like this. Gonna need to check this out then go back in history and re-commit those commits (specifically the one by copilot with the co-authored-by line) to not contain that.
Also I disagree with the changes it proposed (which you applied) to the workshop mod handling.
Add fedora support. Fixes issue #166