Skip to content
This repository was archived by the owner on Jan 21, 2025. It is now read-only.

clean --all#265

Merged
mordamax merged 5 commits intomasterfrom
mak-bot-clean-addon
Feb 7, 2024
Merged

clean --all#265
mordamax merged 5 commits intomasterfrom
mak-bot-clean-addon

Conversation

@mordamax
Copy link
Contributor

@mordamax mordamax commented Feb 6, 2024

Closes #167

  • added bot clean --all which cleans also humans' command bot requests
  • added type_boolean into schema, for such bool flags with explanation possible to set for all kinds of flags

@mordamax mordamax requested a review from a team as a code owner February 6, 2024 22:20
const isOriginComment = comment.id === originalComment.id;
// testing each comment with real commander and pulling repos (even cached) is quite expensive
// so we just check if comment has the command pattern, assuming that if it includes pattern, it's a request to bot
const commandPattern = new RegExp(`^${config.botPullRequestCommentMention} .*$`, "i");

Check failure

Code scanning / CodeQL

Regular expression injection

This regular expression is constructed from a [environment variable](1).
Copy link
Contributor

@rzadp rzadp left a comment

Choose a reason for hiding this comment

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

LGTM.

@mordamax mordamax merged commit 3bbce12 into master Feb 7, 2024
@mordamax mordamax deleted the mak-bot-clean-addon branch February 7, 2024 16:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bot clean: all including the human messages interacting with bot

2 participants