docs: Longshot's save tool is now longshot_saves.py, and its repository saves exception #69

Merged

HumanBean merged 1 commit from HumanBean/LongshotLauncher:docs/longshot-saves into main

2026-11-14 21:43:19 +00:00

HumanBean commented

2026-11-14 18:56:00 +00:00

Owner

Longshot renamed its save tool from scripts/oot_saves.py to scripts/longshot_saves.py, and proposes a narrow exception to the no-saves rule for scrubbed saves in its assets/saves/ (MagicBeanDev/Longshot#543). The launcher's comments and docs name that script, and its CONTRIBUTING.md repeats the rule, so both should follow.

What changed (comments and docs only, no code):

Core/Saves.cs and tests/launcher_core/SavesChecks.cs comments, README.md and docs/saves.md: oot_saves is now longshot_saves wherever Longshot's script is meant. Where Hookshot's own scripts/oot_saves.py is meant (its source, the credits), the old name stays.
CONTRIBUTING.md: one sentence saying Longshot's scrubbed repo saves are the one exception and live in Longshot, never here.

Tested: the diff touches only comments and Markdown. No local build was run (to keep CPU free); CI compiles it.

Merge after MagicBeanDev/Longshot#543, which this follows.

Longshot renamed its save tool from `scripts/oot_saves.py` to `scripts/longshot_saves.py`, and proposes a narrow exception to the no-saves rule for scrubbed saves in its `assets/saves/` (MagicBeanDev/Longshot#543). The launcher's comments and docs name that script, and its CONTRIBUTING.md repeats the rule, so both should follow. **What changed** (comments and docs only, no code): - `Core/Saves.cs` and `tests/launcher_core/SavesChecks.cs` comments, `README.md` and `docs/saves.md`: `oot_saves` is now `longshot_saves` wherever Longshot's script is meant. Where Hookshot's own `scripts/oot_saves.py` is meant (its source, the credits), the old name stays. - `CONTRIBUTING.md`: one sentence saying Longshot's scrubbed repo saves are the one exception and live in Longshot, never here. **Tested:** the diff touches only comments and Markdown. No local build was run (to keep CPU free); CI compiles it. **Merge after** MagicBeanDev/Longshot#543, which this follows.

HumanBean added 1 commit

2026-11-14 18:56:01 +00:00

docs: Longshot's save tool is now longshot_saves.py, and its repository saves exception

CI / Compile and test (pull_request) Has been skipped

Details

CI / Conventional Commit title (pull_request) Successful in 6s

Details

388bc89d98

Longshot renamed scripts/oot_saves.py to scripts/longshot_saves.py (and its
test), and proposes scrubbed saves in its assets/saves/ (Longshot #543).
The launcher's comments, docs and README now name the new script, and
CONTRIBUTING.md notes that the exception lives in Longshot, never here.
Hookshot's own scripts/oot_saves.py keeps its name where Hookshot is meant.
Comments and docs only: no code changes.

Assisted-by: Claude Code
Co-Authored-By: Claude Opus 5.5 <[email protected]>

HumanBean changed title from ~~WIP: docs: Longshot's save tool is now longshot_saves.py, and its repository saves exception~~ to docs: Longshot's save tool is now longshot_saves.py, and its repository saves exception

2026-11-14 21:43:09 +00:00

HumanBean merged commit 3eeaf02b9d into main

2026-11-14 21:43:19 +00:00

HumanBean referenced this pull request from a commit

2026-11-14 21:43:19 +00:00

docs: Longshot's save tool is now longshot_saves.py, and its repository saves exception (#69)

HumanBean deleted branch docs/longshot-saves

2026-11-14 21:43:19 +00:00