Repository navigation
Add toolchain like features - #7
Draft
UnsignedArduino wants to merge 16 commits into
Draft
UnsignedArduino wants to merge 16 commits into
UnsignedArduino wants to merge 16 commits into
Conversation
UnsignedArduino
marked this pull request as draft
August 1, 2026 00:15
Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01Mbb9ZcpdvCZoLyyheJd2FF
Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01Mbb9ZcpdvCZoLyyheJd2FF
- Add the required `target:` to the e2e fixture config. - Install the toolchain and target in a session fixture before building. - Remove `cache=False` so builds are incremental. - Check the second build by parsing redun's job status table (every job CACHED, none DONE) instead of wall-clock time. - Update the project context doc, including the newly exposed stale-cache issue for inputs redun can't see. Co-Authored-By: Claude Opus 5.5 <[email protected]>
Drop the main-only base filter on pull_request so stacked PRs (dag, toolchain) get CI. Pushes still only trigger on main. Co-Authored-By: Claude Opus 5.5 <[email protected]>
- MKCD2APP_HOME moves data/state dirs to $MKCD2APP_HOME/{data,state}/<version>.
The e2e tests install into .pytest_cache/d/mkcd2app-home with it, so they
don't touch the real user dirs and reruns hit the install cache.
- Rename `build --clear-cache` to `--full-rebuild` and explain in its help
text that it's how to pick up changes to source files, icons, or GitHub
branches, which aren't tracked automatically.
- Update the project context doc (CI green on all OSes, roadmap, decision
to rely on --full-rebuild for out-of-band inputs).
Co-Authored-By: Claude Opus 5.5 <[email protected]>
- download_sim decodes downloads as UTF-8 instead of res.text (which falls back to ISO-8859-1 without a charset), and all text file reads/writes pass encoding="utf-8" so Windows doesn't use cp1252. - get_resource_template_path(name) now actually joins name. - load_config_from_yaml raises NotImplementedError for a missing/unknown version instead of AttributeError. - Fix uninstall_toolchain and fill_website_template docstrings, drop a stray comment in "Racers offline.yaml", and update the project context doc. Co-Authored-By: Claude Opus 5.5 <[email protected]>
…file install_deps_and_build_website* hashed the whole filled site, so any game or asset change re-ran npm ci. install_website_deps now runs npm ci once from only the template's package.json and package-lock.json, and the renamed build_website* tasks copy that node_modules into their own site copy and run only npm run build*. static and static-singlefile share the one install. Co-Authored-By: Claude Opus 5.5 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.