Hey folks-

I have a couple (hopefully quick) questions about backporting best
practices (from the lucene/main branch to lucene-solr/branch_8x).
Really appreciate the help!

1. A lot of code reformatting has happened on lucene/main (e.g.,
spotlessApply). When pulling over changes into branch_8x for a given
file, is it preferable to pull in the formatting changes as well, or
are we trying to leave the formatting of branch_8x alone as much as
possible? It generally seems easier to pull the formatting changes
over as well, but it can make a PR look more involved than it really
is.

2. In a case where multiple small changes were made to the same area
of code (e.g., a fast-follow update or a bug fix), is it acceptable to
combine those changes into one PR against branch_8x? Or do we want to
maintain a more strict 1:1 relationship between a change on main and
branch_8x? Seems like it would be OK to combine a couple small changes
into a single backport PR, but curious if people feel differently.

Thanks everyone!

Cheers,
-Greg

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to