Skip to content

Record the state-name and bypass deviations, bring in starter files - #8

Merged
MaymiPR merged 1 commit into
mainfrom
chore/standard-followups
Nov 11, 2026
Merged

MaymiPR merged 1 commit into
mainfrom
chore/standard-followups

Conversation

@MaymiPR

@MaymiPR MaymiPR commented Nov 11, 2026

Copy link
Copy Markdown
Owner
  • CONTRIBUTING, Deviations from the standard (decided 2026-11-11): the state tag stays STATE and the editor property uiWidth, which saved projects read; no bypass parameter of its own, the JUCE VST3 wrapper's bypass serves.
  • tools/Snapshot.cpp and shared/ui/Widgets.h from the starter (--option to SnapshotHooks::snapshotOption); scripts/build_native.py from the starter; STANDARDS.md and docs/screenshots/README.md synced. tools/HostTest.cpp already equals the starter's. The sampler keeps its own make_screenshots.py (its snapshot tool takes a sample), as the standard now allows.
  • tests/data/state-1.2.0.txt unchanged from main.

Checked locally: scripts/check.ps1 -KeepFl with every tracked source touched (build ok, 0 warnings, 5/5 ctest, render regression ok); check_repo.py 0 findings; check_consistency.py --public --history 0 FAIL; sync_template.py --check 0 differences; frozen identity unchanged.

🤖 Generated with Claude Code

CONTRIBUTING records under the deviations that the state tag stays
STATE and the editor's size property uiWidth, which saved projects
read, and that the plug-in has no bypass parameter of its own: the
JUCE VST3 wrapper's bypass serves. tools/Snapshot.cpp hands any further
--option to SnapshotHooks::snapshotOption, which shared/ui/Widgets.h
now declares; scripts/build_native.py is the starter's. STANDARDS.md
and docs/screenshots/README.md follow the starter. The sampler keeps
its own make_screenshots.py, as its snapshot tool takes a sample.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@MaymiPR
MaymiPR merged commit 900705e into main Nov 11, 2026
5 checks passed
@MaymiPR
MaymiPR deleted the chore/standard-followups branch November 11, 2026 18:57
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