Skip to content

Added option to filter out a snippet of the replacement payload using the same regex.#21

Merged
nefrob merged 8 commits intonefrob:masterfrom
Aukevanoost:master
Oct 20, 2024
Merged

Added option to filter out a snippet of the replacement payload using the same regex.#21
nefrob merged 8 commits intonefrob:masterfrom
Aukevanoost:master

Conversation

@Aukevanoost
Copy link
Copy Markdown
Contributor

Description

This PR allows for Regex in the payload as well, for example if you only need a part of the input file you're reading.

No breaking changes, only an optional flag that will filter out the regex from the replacement payload and only update the description with the (first) match of the regex.

@Aukevanoost Aukevanoost changed the title Added option to filter out a snippet of the replacement payload instead of the full payload. Added option to filter out a snippet of the replacement payload using the same regex. Oct 14, 2024
Copy link
Copy Markdown
Owner

@nefrob nefrob left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

I'm wary of adding support for transformations of the input when the main goal of this action is to provide simple transformation of a PR body.

That being said, this is fairly simple like the rest of this action and could provide value for basic use cases.

Comment thread index.js Outdated
Comment thread index.js
@Aukevanoost Aukevanoost requested a review from nefrob October 16, 2024 05:06
Copy link
Copy Markdown
Owner

@nefrob nefrob left a comment

Choose a reason for hiding this comment

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

LGTM. I'll flup with some general cleanup.

@nefrob nefrob merged commit cda639f into nefrob:master Oct 20, 2024
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