mosabab commented on pull request #1097: URL: https://github.com/apache/cordova-android/pull/1097#issuecomment-808140596
> I want this single PR to be split up into two or three separate PRs. > > 1. `feat: upgrade [email protected]` > 2. `feat: upgrade [email protected]` > 3. `feat: upgrade [email protected]` > > We generally squash merge PRs. Also if we need to revert a change because of issues, we will revert the entire PR. This means everything will be dropped. > > IMO, seeing that these changes only updates to the latest minor release, it should be safe to be in the next minor release. > > Also, I do not see why this PR contains code changes in `CordovaWebViewImpl.java` for this type of PR. It should be removed. I suspect because your not creating branches for your work your mixing to many things. Thanks for explain! Done -- 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]
