Skip to content

docs: remove duplicate words.#51215

Closed
JeanMeche wants to merge 1 commit intoangular:mainfrom
JeanMeche:docs/the-typo
Closed

docs: remove duplicate words.#51215
JeanMeche wants to merge 1 commit intoangular:mainfrom
JeanMeche:docs/the-typo

Conversation

@JeanMeche
Copy link
Copy Markdown
Member

@JeanMeche JeanMeche commented Jul 31, 2023 •
edited
Loading

Using the \b(\w+)\s+\1\b we can find duplicate words. Let's remove them so we don't get more PRs with this simple changes.

@JeanMeche
docs: remove duplicate words.
014f70d 
Using the `\b(\w+)\s+\1\b` we can find duplicate word. Let's remove them.
@ngbot ngbot bot added this to the Backlog milestone Jul 31, 2023
@JeanMeche JeanMeche marked this pull request as ready for review July 31, 2023 11:56
@pullapprove pullapprove bot requested review from AndrewKushnir and alxhub July 31, 2023 11:56
AndrewKushnir
AndrewKushnir approved these changes Jul 31, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@AndrewKushnir AndrewKushnir left a comment

Choose a reason for hiding this comment

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

@JeanMeche 👍

@AndrewKushnir AndrewKushnir added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Jul 31, 2023
@AndrewKushnir AndrewKushnir removed the request for review from alxhub July 31, 2023 17:57
@ngbot
Copy link
Copy Markdown

ngbot bot commented Jul 31, 2023

I see that you just added the action: merge label, but the following checks are still failing:
    failure status "google-internal-tests" is failing
    pending status "pullapprove" is pending
    pending 1 pending code review

If you want your PR to be merged, it has to pass all the CI checks.

If you can't get the PR to a green state due to flakes or broken main, please try rebasing to main and/or restarting the CI job. If that fails and you believe that the issue is not due to your change, please contact the caretaker and ask for help.

@pullapprove pullapprove bot requested a review from alxhub July 31, 2023 17:57
@AndrewKushnir AndrewKushnir added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label Jul 31, 2023
@AndrewKushnir
Copy link
Copy Markdown
Contributor

AndrewKushnir commented Jul 31, 2023

Caretaker note: low risk change (docs-only), no presubmit or extra reviews are required.

This was referenced Aug 1, 2023
Closed
Closed
Closed
@alxhub alxhub added target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Aug 1, 2023
@alxhub
Copy link
Copy Markdown
Member

alxhub commented Aug 1, 2023

This PR was merged into the repository by commit a871e23.

@alxhub alxhub closed this in a871e23 Aug 1, 2023
@JeanMeche JeanMeche deleted the docs/the-typo branch August 3, 2023 13:37
@angular-automatic-lock-bot
Copy link
Copy Markdown

angular-automatic-lock-bot bot commented Sep 3, 2023

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 3, 2023
ChellappanRajan pushed a commit to ChellappanRajan/angular that referenced this pull request Jan 23, 2024
@JeanMeche @ChellappanRajan
docs: remove duplicate words. (angular#51215)
f1f4d8f 
Using the `\b(\w+)\s+\1\b` we can find duplicate word. Let's remove them.

PR Close angular#51215
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@AndrewKushnir AndrewKushnir AndrewKushnir approved these changes

@alxhub alxhub Awaiting requested review from alxhub

Assignees

No one assigned

Labels

action: merge The PR is ready for merge by the caretaker merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: minor This PR is targeted for the next minor release

Projects

None yet

Milestone

Backlog

Development

Successfully merging this pull request may close these issues.

3 participants

@JeanMeche @AndrewKushnir @alxhub