mbien commented on PR #5238: URL: https://github.com/apache/netbeans/pull/5238#issuecomment-1374843378
@tbw777 doing larger PRs is fine. Just not crazy large. Someone has to review the PR somehow. So it depends how complex a PR is. If it does something simple in 200 files.. it might be ok. You can also put multiple commits into a single PR which makes reviewing also easier. This one for example is too big #5203 since merging this will cause too many conflicts at once to other PRs. This is better #5189. Also keep in mind that PRs uses apache resources since CI has to build and test a lot - too many PRs is also bad if they can be combined into medium-sized PRs. So please don't open too many PRs since we have to merge some first. You can work with your clone of netbeans without creating PRs, and once we can merge more you simply open more if you want. It can take some time. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
