Skip to content

fix: escape apostrophes in fingerprint globs#2875

Open
cyphercodes wants to merge 1 commit into
go-task:mainfrom
cyphercodes:fix/apostrophe-path-generates
Open

fix: escape apostrophes in fingerprint globs#2875
cyphercodes wants to merge 1 commit into
go-task:mainfrom
cyphercodes:fix/apostrophe-path-generates

Conversation

@cyphercodes
Copy link
Copy Markdown

Summary

  • Treat apostrophes in fingerprint glob paths as literal path characters before shell-style field expansion.
  • Add a regression test that runs a Taskfile from a temporary directory whose name contains an apostrophe, with both sources and generates configured.

Fixes #2874.

Tests

  • go test . -run TestStatusChecksumAllowsApostropheInWorkingDirectory -count=1
  • go test ./internal/fingerprint ./internal/execext . -count=1
  • go test ./...
  • git diff --check

AI assistance was used to help investigate and prepare this patch; I reviewed and tested the changes.

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.

"reached EOF without closing quote" when folder name has a quote in it

1 participant