magibney commented on pull request #2470: URL: https://github.com/apache/lucene-solr/pull/2470#issuecomment-802986119
I'm inclined to leave this as-is for now (with the recent minor addition of the `merge.renameLimit` increase, which I think is probably mostly cosmetic wrt suppressing a warning that I think in this context is probably not significant? -- can't hurt, in any case). ... unless there are any modifications suggested. My impression is that this is placed in the root of `lucene-solr` not so much to catch eyes as because this is really about the migration, for people who may still be looking mainly at the legacy joint project, and because it applies equally to Lucene- and Solr-focused PRs. I could see placing a more general-purpose "guide to working with Solr PRs" markdown/adoc file in `solr:solr/dev-docs`, and maybe linking from there to this top-level legacy project "PR migration" readme? That way the content can be maintained in a single (most logical) place. However this is approached, perhaps it makes sense as a separate issue (in the new solr TLP)? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
