Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Escape page path when generating RegExp to find ancestors children #9550

Merged
merged 7 commits into from
Jan 17, 2025

Conversation

yuki-takei
Copy link
Member

Copy link

changeset-bot bot commented Jan 16, 2025

🦋 Changeset detected

Latest commit: 2aba836

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@growi/core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@yuki-takei yuki-takei changed the title fix: Escape page path when generating RegExp to find ancestorschildren fix: Escape page path when generating RegExp to find ancestors children Jan 17, 2025
@yuki-takei
Copy link
Member Author

@mergify queue

Copy link
Contributor

mergify bot commented Jan 17, 2025

queue

🛑 The pull request has been removed from the queue default

The merge conditions cannot be satisfied due to failing checks.

You can take a look at Queue: Embarked in merge queue check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

mergify bot added a commit that referenced this pull request Jan 17, 2025
Copy link
Contributor

mergify bot commented Jan 17, 2025

This pull request has been removed from the queue for the following reason: checks failed.

The merge conditions cannot be satisfied due to failing checks:

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.

If you want to requeue this pull request, you need to post a comment with the text: @mergifyio requeue

@yuki-takei
Copy link
Member Author

@mergify dequeue

Copy link
Contributor

mergify bot commented Jan 17, 2025

dequeue

☑️ The pull request is not queued

@yuki-takei
Copy link
Member Author

@mergify queue

Copy link
Contributor

mergify bot commented Jan 17, 2025

queue

🛑 The pull request has been removed from the queue default

The merge conditions cannot be satisfied due to failing checks.

You can take a look at Queue: Embarked in merge queue check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

@yuki-takei
Copy link
Member Author

@mergify requeue

Copy link
Contributor

mergify bot commented Jan 17, 2025

requeue

❌ This pull request head commit has not been previously disembarked from queue.

@yuki-takei
Copy link
Member Author

@mergify queue

Copy link
Contributor

mergify bot commented Jan 17, 2025

queue

🛑 The pull request has been removed from the queue default

The merge conditions cannot be satisfied due to failing checks.

You can take a look at Queue: Embarked in merge queue check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

mergify bot added a commit that referenced this pull request Jan 17, 2025
Copy link
Contributor

mergify bot commented Jan 17, 2025

This pull request has been removed from the queue for the following reason: checks failed.

The merge conditions cannot be satisfied due to failing checks:

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.

If you want to requeue this pull request, you need to post a comment with the text: @mergifyio requeue

@yuki-takei
Copy link
Member Author

@mergify queue

Copy link
Contributor

mergify bot commented Jan 17, 2025

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at ffa0fef

mergify bot added a commit that referenced this pull request Jan 17, 2025
@mergify mergify bot merged commit ffa0fef into master Jan 17, 2025
18 checks passed
@mergify mergify bot deleted the fix/escape-regexp-when-finding-ancestors-children branch January 17, 2025 12:11
This was referenced Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

1 participant