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

Remove "THIS CODE IS GENERATED" comments 1 of 12 #263

Closed
wants to merge 1 commit into from

Conversation

AarjavP
Copy link

@AarjavP AarjavP commented Nov 14, 2022

same as #235 but broken down into different commits / branches.

Please let me know how / when the changelog should be submitted.

I am creating just this 1 PR right now, but I have pushed other branches to my fork here: https://github.com/AarjavP/opensearch-java/branches
Most have < 200 files, and the one with most files has 213. If this is ok I can go ahead and create rest of the PRs.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

broken down into multiple commits as per opensearch-project#235

Signed-off-by: AarjavP <aarjavpatel@gmail.com>
@VachaShah
Copy link
Collaborator

Hi @AarjavP, sure this looks good!

@VachaShah
Copy link
Collaborator

@AarjavP You will have to add Changelog to all your PRs.

@dblock
Copy link
Member

dblock commented Nov 15, 2022

@VachaShah You sure you want 12 (!) PRs instead of one big one? What's the advantage? Let's just merge #235, I am A-OK on that.

@AarjavP
Copy link
Author

AarjavP commented Nov 15, 2022

How about I push the 11 other commits into this PR and update the change log here only. I will close other 11 open PRs.

This way we can also avoid merge conflicts and merge commits due to change log. For review you can select one commit at a time instead of all at once.

@VachaShah
Copy link
Collaborator

@AarjavP still looking to finish this?

@VachaShah
Copy link
Collaborator

Done via #598

@VachaShah VachaShah closed this Aug 15, 2023
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.

3 participants