breautek commented on issue #189: Variable for Require GPS Hardware URL: https://github.com/apache/cordova-plugin-geolocation/pull/189#issuecomment-597178340 > why isn't this merged ? this or #171 @breautek Two main reasons: 1. Because both PRs introduce preference in a `UPPERCASE_FORMAT`, which doesn't match existing format cordova uses for preferences, which in a `PascalCase` format. I'm hesitate to add inconsitencies. 2. Generally speaking, we need at least two positive votes and 0 negative votes from apache members for a PR to be accepted. So even if I agreed with this PR, I need one more member to review before I can merge.
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
