Skip to content

MINOR:Move TROGDOR.md to trogdor module#13979

Merged
divijvaidya merged 2 commits into
apache:trunkfrom
hudeqi:fix_trogdor
Jul 10, 2023
Merged

MINOR:Move TROGDOR.md to trogdor module#13979
divijvaidya merged 2 commits into
apache:trunkfrom
hudeqi:fix_trogdor

Conversation

@hudeqi

@hudeqi hudeqi commented Jul 8, 2023

Copy link
Copy Markdown
Contributor

move TROGDOR.md in root dir to trogdor module, and rename to README.md

@divijvaidya
divijvaidya self-requested a review July 9, 2023 13:46
@divijvaidya

Copy link
Copy Markdown
Member

I accidentally approved. Please ignore that.

We need a lot many changed. If you search the code base, there are many references to TROGDOR.md, we need to change all of them. Also, it's referenced in build.gradle and

@hudeqi

hudeqi commented Jul 10, 2023

Copy link
Copy Markdown
Contributor Author

I accidentally approved. Please ignore that.

We need a lot many changed. If you search the code base, there are many references to TROGDOR.md, we need to change all of them. Also, it's referenced in build.gradle and

sorry, I ignore that, and I have updated. @divijvaidya

@divijvaidya divijvaidya left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for the change. LGTM!

@divijvaidya
divijvaidya merged commit 8be601d into apache:trunk Jul 10, 2023
@hudeqi
hudeqi deleted the fix_trogdor branch July 10, 2023 23:10
Cerchie pushed a commit to Cerchie/kafka that referenced this pull request Jul 25, 2023
Reviewers: Divij Vaidya <diviv@amazon.com>

---------

Co-authored-by: Deqi Hu <deqi.hu@shopee.com>
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