Github user daserge commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/131#issuecomment-156454285
@convey-mane, I believe you should do the following with the PR:
- Send a message to the [dev-list](http://cordova.apache.org/contact/) with
the feature proposal for discussion ([here is an example of such a
thread](http://markmail.org/thread/6jjxas3wrhwl6zb5)),
- If the vote is positive, then create a new issue on
http://issues.cordova.io and assign the id you get to this pull request, so
it's linked and tracked. Example: PR and commits title `CB-9999 iOS: new
resizable cropping rectangle instead of Apple's standard square` (this enables
Jira&Github integration and issue tracking),
- Squash commits into one or two (f.e. split dependency library into a
separate commit),
- Rebase the PR to master to avoid merge conflicts.
You can force-push into your branch to remove unnecessary/old commits (off
course don't forget to back-up your work).
IMO it is better to use rebase vs merge in terms of history readability.
Useful link:
https://github.com/phonegap/phonegap/wiki/Git-Contributor-Workflow
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org
