Repository navigation
opensgolf: Add version 0.5.6 - #1840
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VX9jWo73YE2zmxz73HLhbR
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Package installation and updates bucket/opensgolf.json |
The manifest defines 64-bit and ARM64 release downloads, hashes, and extraction directories. It exposes opensgolf.exe as a command and Start-menu shortcut, and configures GitHub version checks and SHA256SUMS-based hash lookup for autoupdates. |
Priority:
Merge Risk: 
3915b
-
No actionable installation or update issue was established; the manifest is ready to merge after normal checks.
Pre-merge checks: 
-
Check name Status Explanation Title check The title clearly identifies the package and the version being added. It matches the main change. Docstring Coverage No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Description check The pull request description provides a clear summary, project homepage, package details, related issue, and contributing-guide confirmation. It includes the required issue reference and checklist. Th… Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
Full details: Description check
-
Explanation
The pull request description provides a clear summary, project homepage, package details, related issue, and contributing-guide confirmation. It includes the required issue reference and checklist. The checkbox spacing is nonstandard, but the intent is clear.
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
| "description": "Open source engine for the golf course building game SimGolf, with its own free graphics and sounds", | ||
| "homepage": "https://github.com/crspytopgn/opensgolf", | ||
| "license": "GPL-2.0-or-later", | ||
| "notes": "Saves and settings are in %APPDATA%\\OpenSGolf. Update with 'scoop update opensgolf' rather than the game's own Update now, so that Scoop knows the version installed.", |
There was a problem hiding this comment.
I suggest disabling in-app automatic updates and update checks/buttons when installing the game via Scoop. This can be done by writing a default configuration file that disables the game's own update mechanism in a post_install step.
Additionally, consider supporting a portable mode (e.g. by creating an empty file named portable.txt next to the executable, and writing this file in post_install) as Scoop apps are meant to be portable by default.
Either way, this can be done in a future update.
Adds OpenSGolf, a free, open source (GPL-2.0-or-later) engine for the golf course building game SimGolf, with its own free graphics and sounds. Homepage: https://github.com/crspytopgn/opensgolf
64bit and arm64 portable zips from the GitHub releases.
checkver: github, with autoupdate hashes read from the release's SHA256SUMS.
Closes #1841