Skip to content

Added depreciation note in npm-scripts.md for prepublish#268

Closed
mkotsollaris wants to merge 1 commit intonpm:latestfrom
mkotsollaris:patch-1
Closed

Added depreciation note in npm-scripts.md for prepublish#268
mkotsollaris wants to merge 1 commit intonpm:latestfrom
mkotsollaris:patch-1

Conversation

@mkotsollaris
Copy link
Copy Markdown
Contributor

No description provided.

@mkotsollaris mkotsollaris requested a review from a team as a code owner October 16, 2019 06:48
@claudiahdz claudiahdz added semver:patch semver patch level for changes Release 6.x work is associated with a specific npm 6 release labels Nov 12, 2019
claudiahdz pushed a commit that referenced this pull request Nov 13, 2019
claudiahdz pushed a commit that referenced this pull request Nov 13, 2019
claudiahdz pushed a commit that referenced this pull request Nov 13, 2019
claudiahdz pushed a commit that referenced this pull request Nov 13, 2019
claudiahdz pushed a commit that referenced this pull request Nov 13, 2019
@claudiahdz claudiahdz mentioned this pull request Nov 18, 2019
@mkotsollaris mkotsollaris deleted the patch-1 branch November 18, 2019 23:03
Jah-yee pushed a commit to Jah-yee/cli that referenced this pull request Apr 16, 2026
…#412)

* docs: warn about zsh \! history expansion in sheet range examples

Add a "Shell Tips" section to the gws-shared skill explaining that
zsh interprets \! inside single quotes as history expansion, which
mangles sheet ranges like 'Sheet1\!A1:B2'.

Replace single quotes with double quotes around sheet ranges
containing \! in:
- registry/recipes.yaml (source for generated recipe skills)
- src/helpers/sheets.rs (+read after_help examples)
- src/generate_skills.rs (shared skill template)
- Generated SKILL.md files for affected recipes and gws-sheets-read

Closes npm#268

* fix: escape double quotes in after_help string literal

The after_help string for +read uses a regular string literal, not a
raw string, so inner double quotes must be escaped with backslashes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release 6.x work is associated with a specific npm 6 release semver:patch semver patch level for changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants