Conversation
The CLI has defaulted to green.yml since read-state started picking the reader from the file extension, but the README and the specification still described desired state as EDN and printed green.edn in the usage line. - README: "desired state in YAML or EDN", plus a bullet covering the default file, the extension-based reader, and the COLORS_PAR_* channel. - index.html: correct the philosophy bullet and the section 2 lead, add a reader table, and fix the usage synopsis and examples in section 10. - index.html: document the COLORS_PAR_* secret channel, which arrived with the YAML reader and was never written up, and add :green/state-file to the reserved-key table. - index.html: add read-state, read-pars and par-name to the API index, list the four namespaces missing from the layout tree, note yamlstar in the deps, and cover cli/yaml/process in the testing section. - index.html: say plainly that the in-repo examples stay on green.edn while downstream packages use colors.yml, so both halves stay exercised. EDN is still read for any non-.yml/.yaml extension, so the docs describe a default rather than a replacement. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CJ3gwMLE1pmNLDPgbtECYg
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.
The CLI has defaulted to green.yml since read-state started picking the
reader from the file extension, but the README and the specification still
described desired state as EDN and printed green.edn in the usage line.
default file, the extension-based reader, and the COLORS_PAR_* channel.
reader table, and fix the usage synopsis and examples in section 10.
the YAML reader and was never written up, and add :green/state-file to
the reserved-key table.
the four namespaces missing from the layout tree, note yamlstar in the
deps, and cover cli/yaml/process in the testing section.
downstream packages use colors.yml, so both halves stay exercised.
EDN is still read for any non-.yml/.yaml extension, so the docs describe a
default rather than a replacement.
Co-Authored-By: Claude Opus 5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01CJ3gwMLE1pmNLDPgbtECYg