Skip to content

docs(spec): misc cleanups — whitespace, capitalization, three-cmd glob#201

Merged
CGMossa merged 1 commit into
mainfrom
docs/specs-misc-cleanups
May 17, 2026
Merged

docs(spec): misc cleanups — whitespace, capitalization, three-cmd glob#201
CGMossa merged 1 commit into
mainfrom
docs/specs-misc-cleanups

Conversation

@CGMossa
Copy link
Copy Markdown
Contributor

@CGMossa CGMossa commented May 16, 2026

Minor

AI-written details

Summary

Small spec edits with no behavioral content, carved out of #149:

Where Before After Why
L29 `init` will create the `dvs.toml`.␣ (trailing space) trailing space stripped trailing whitespace
L36 add each file to a .gitignore add each metadata file to .gitignore that are adjacent to the files matches actual implementation in dvs::gitignore::* — the metadata file is what's added
L50 double blank line before ## In-depth spec single blank line normalization
L155 #### R Package #### R package other three command sections already use lowercase "package"
L351 `add` and `get` both accept a `--glob` flag `add`, `status`, `get` accept a `--glob` flag status also accepts --glob; "both" is also wrong for three operations

Test plan

  • specs.md renders cleanly on GitHub
  • git grep -n 'R Package' returns no occurrences in specs.md
  • dvs status --help confirms --glob exists

Drafted by Claude (claude-opus-4-7). Reviewed by the author.

Collected small spec edits with no behavioral content:

- L29 strip trailing space on `\`init\` will create the \`dvs.toml\`. `
- L36 fix the gitignore step: it's the metadata file (and adjacent
  data file) being added, not "each file" — matches the actual
  implementation in dvs::gitignore::*
- L50 drop double blank line before `## In-depth spec`
- L155 capitalization `#### R Package` → `#### R package` (the
  other three sections already use lowercase "package")
- L351 globbing section: `\`add\` and \`get\` both accept` →
  `\`add\`, \`status\`, \`get\` accept` (status accepts --glob too;
  "both" is also wrong for three operations)

Carved out of #149.
CGMossa added a commit that referenced this pull request May 16, 2026
@CGMossa CGMossa merged commit 7a8e308 into main May 17, 2026
1 check passed
@CGMossa CGMossa deleted the docs/specs-misc-cleanups branch May 17, 2026 09:02
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