Skip to content

Write pip extras with double quotes: single quotes break in Windows cmd (v3.1.20) - #87

Merged
click0 merged 1 commit into
mainfrom
claude/refactor-data-masking-lIcWN
Oct 17, 2026
Merged

click0 merged 1 commit into
mainfrom
claude/refactor-data-masking-lIcWN

Conversation

@click0

@click0 click0 commented Oct 17, 2026

Copy link
Copy Markdown
Owner

Summary

The maintainer questioned pip install '.[full]' in the README. The syntax itself is right — pip accepts a local path with extras, and a fresh venv installs the checkout with cryptography and pyyaml from it — but in Windows cmd.exe single quotes are not quoting characters: pip receives '.[full]' literally and fails with "Invalid requirement". Double quotes work in bash, zsh, PowerShell and cmd alike.

  • README (EN/UK), docs/INSTALL.md, requirements.txt, docs/config-examples/README.md, the CLI hints for --encrypt (pip install "data-masking[security]"), .github/workflows/ci.yml and CLAUDE.md use double quotes for every extras spec.
  • The README install snippet starts with cd data-masking so it is clear the command runs from the repository root.
  • Version 3.1.20, CHANGELOG entry.

Verification

  • pytest: 1533 passed, 1 skipped; mypy 0 errors; flake8 clean (the CLI module imports — the hint strings were re-quoted by hand).
  • pip install ".[full]" from the checkout in a fresh venv: installs 3.1.x with both extras, data-mask --version works.

Generated by Claude Code

…md (v3.1.20)

pip install '.[full]' is valid pip syntax and installs the checkout with
the extra (verified in a fresh venv), but in Windows cmd.exe single quotes
are not quoting characters: pip receives the quotes literally and fails
with "Invalid requirement". Double quotes work in bash, zsh, PowerShell
and cmd alike, so README (EN/UK), INSTALL.md, requirements.txt, the
config-examples README, the CLI hints for --encrypt, CI and CLAUDE.md use
them. The README install snippet starts with cd into the checkout.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XT6iUWaQgahXDB9TWX9Bq7
Generated-With: Claude Code 2.1.42
@click0
click0 merged commit f03b227 into main Oct 17, 2026
16 checks passed
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.

2 participants