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.

Sign up for GitHub

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

Jump to bottom

trivial word dedup #22508

Merged
merged 2 commits into from
Jun 1, 2023
Merged

trivial word dedup #22508

merged 2 commits into from
Jun 1, 2023

Conversation

rbowen
Copy link
Contributor

@rbowen rbowen commented May 2, 2023

What is the purpose of the change

Trivial word deduplication: "and and" and "the the"

@flinkbot
Copy link
Collaborator

flinkbot commented May 2, 2023 •
edited

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

@mbalassi mbalassi self-requested a review May 6, 2023 10:50
mbalassi
mbalassi requested changes May 6, 2023
View reviewed changes
Copy link
Contributor

@mbalassi mbalassi left a comment

Choose a reason for hiding this comment

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

Thanks, @rbowen. Since you touched some of the source files please update your PR to satisfy the checkstyle rules by running:

mvn spotless:apply

And committing the diff.

@mbalassi mbalassi self-assigned this May 6, 2023
@mbalassi
Copy link
Contributor

@rbowen let me know if you prefer that I do the amendment upon merging, that is fine too. 😏

@rbowen
Copy link
Contributor Author

rbowen commented May 31, 2023

Oops. Sorry, missed the earlier note. Taking care of it now.

@rbowen
Copy link
Contributor Author

rbowen commented May 31, 2023

Done.

@mbalassi mbalassi self-requested a review June 1, 2023 12:51
mbalassi
mbalassi approved these changes Jun 1, 2023
View reviewed changes
Copy link
Contributor

@mbalassi mbalassi left a comment

Choose a reason for hiding this comment

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

LGTM

@mbalassi mbalassi merged commit 7457251 into apache:master Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mbalassi mbalassi mbalassi approved these changes

Assignees

@mbalassi mbalassi

Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@rbowen @flinkbot @mbalassi