Skip to content

Added description of when files promisers are interpreted as regular expressions#3745

Open
nickanderson wants to merge 1 commit into
cfengine:masterfrom
nickanderson:docs-promiser-regex-detection
Open

Added description of when files promisers are interpreted as regular expressions#3745
nickanderson wants to merge 1 commit into
cfengine:masterfrom
nickanderson:docs-promiser-regex-detection

Conversation

@nickanderson

Copy link
Copy Markdown
Member

Files promises by default guess if they are meant to be interpreted as a regular
expression or not and this adds explicit description of what that guess is based
on.

@nickanderson
nickanderson force-pushed the docs-promiser-regex-detection branch from 9d1ebc1 to 8901399 Compare July 23, 2026 18:02
character literal). So `/tmp/test.txt` matches only `/tmp/test.txt`, not
`/tmp/test-txt`.

When regular expression is chosen the entire promiser is converted into a

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How do you choose regular expression?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here, I simplified it what do you think now

@olehermanse olehermanse left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not clear to me what you mean by choosing regular expression.

…expressions

Files promises by default guess if they are meant to be interpreted as a regular
expression or not and this adds explicit description of what that guess is based
on.
@nickanderson
nickanderson force-pushed the docs-promiser-regex-detection branch from dbd8fdf to 4d05c6e Compare July 24, 2026 19:29
@nickanderson
nickanderson requested a review from olehermanse July 24, 2026 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants