-
-
Notifications
You must be signed in to change notification settings - Fork 46
docs: update readme + contribution guidelines #89
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
<!-- | ||
We, the rest of the NativeScript community, thank you for your | ||
contribution! | ||
To help the rest of the community review your change, please follow the instructions in the template. | ||
--> | ||
|
||
<!-- PULL REQUEST TEMPLATE --> | ||
<!-- (Update "[ ]" to "[x]" to check a box) --> | ||
|
||
## PR Checklist | ||
|
||
- [ ] The PR title follows our guidelines: https://github.com/NativeScript/NativeScript/blob/master/CONTRIBUTING.md#commit-messages. | ||
- [ ] There is an issue for the bug/feature this PR is for. To avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it. | ||
- [ ] All existing tests are passing | ||
- [ ] Tests for the changes are included | ||
|
||
## What is the current behavior? | ||
<!-- Please describe the current behavior that you are modifying, or link to a relevant issue. --> | ||
|
||
## What is the new behavior? | ||
<!-- Describe the changes. --> | ||
|
||
Fixes/Implements/Closes #[Issue Number]. | ||
|
||
<!-- If this PR contains a breaking change, please describe the impact and migration path for existing applications below. --> | ||
|
||
<!-- | ||
BREAKING CHANGES: | ||
|
||
|
||
[Describe the impact of the changes here.] | ||
|
||
Migration steps: | ||
[Provide a migration path for existing applications.] | ||
--> | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,75 @@ | ||
# Contributing to NativeScript Imagepicker | ||
|
||
:+1: First of all, thank you for taking the time to contribute! :+1: | ||
|
||
Here are some guides on how to do that: | ||
|
||
<!-- TOC depthFrom:2 --> | ||
|
||
- [Code of Conduct](#code-of-conduct) | ||
- [Reporting Bugs](#reporting-bugs) | ||
- [Requesting Features](#requesting-features) | ||
- [Submitting a PR](#submitting-a-pr) | ||
- [Where to Start](#where-to-start) | ||
|
||
<!-- /TOC --> | ||
|
||
## Code of Conduct | ||
Help us keep a healthy and open community. We expect all participants in this project to adhere to the [NativeScript Code Of Conduct](https://github.com/NativeScript/codeofconduct). | ||
|
||
|
||
## Reporting Bugs | ||
|
||
1. Always update to the most recent master release; the bug may already be resolved. | ||
2. Search for similar issues in the issues list for this repo; it may already be an identified problem. | ||
3. If this is a bug or problem that is clear, simple, and is unlikely to require any discussion -- it is OK to open an issue on GitHub with a reproduction of the bug including workflows and screenshots. If possible, submit a Pull Request with a failing test, entire application or module. If you'd rather take matters into your own hands, fix the bug yourself (jump down to the [Submitting a PR](#submitting-a-pr) section). | ||
|
||
## Requesting Features | ||
|
||
1. Use Github Issues to submit feature requests. | ||
2. First, search for a similar request and extend it if applicable. This way it would be easier for the community to track the features. | ||
3. When requesting a new feature, please provide as much detail as possible about why you need the feature in your apps. We prefer that you explain a need rather than explain a technical solution for it. That might trigger a nice conversation on finding the best and broadest technical solution to a specific need. | ||
|
||
## Submitting a PR | ||
|
||
Before you begin make sure there is an issue for the bug or feature you will be working on. | ||
|
||
Following these steps is the best way to get your code included in the project: | ||
|
||
1. Fork and clone the nativescript-camera repo: | ||
```bash | ||
git clone https://github.com/<your-git-username>/nativescript-camera.git | ||
# Navigate to the newly cloned directory | ||
cd nativescript-camera | ||
# Add an "upstream" remote pointing to the original repo. | ||
git remote add upstream https://github.com/NativeScript/nativescript-camera.git | ||
``` | ||
|
||
2. Read our [development workflow guide](DevelopmentWorkflow.md) for local setup | ||
|
||
3. Create a branch for your PR | ||
```bash | ||
git checkout -b <my-fix-branch> master | ||
``` | ||
|
||
4. The fun part! Make your code changes. Make sure you: | ||
- Follow the [code conventions guide](https://github.com/NativeScript/NativeScript/blob/master/CodingConvention.md). | ||
- Follow the [commit message guidelines](https://github.com/NativeScript/NativeScript/blob/pr-template/CONTRIBUTING.md#commit-messages) | ||
- Update the README if you make changes to the plugin API | ||
|
||
5. Before you submit your PR: | ||
- Rebase your changes to the latest master: `git pull --rebase upstream master`. | ||
- Ensure your changes pass tslint validation. (run `npm run tslint` in the `src` folder). | ||
|
||
6. Push your fork. If you have rebased you might have to use force-push your branch: | ||
``` | ||
git push origin <my-fix-branch> --force | ||
``` | ||
|
||
7. [Submit your pull request](https://github.com/NativeScript/nativescript-camera/compare) and compare to `NativeScript/nativescript-camera`. Please, fill in the Pull Request template - it will help us better understand the PR and increase the chances of it getting merged quickly. | ||
|
||
It's our turn from there on! We will review the PR and discuss changes you might have to make before merging it! Thanks! | ||
|
||
## Where to Start | ||
|
||
If you want to contribute, but you are not sure where to start - look for issues labeled [`help wanted`](https://github.com/NativeScript/nativescript-camera/issues?q=is%3Aopen+is%3Aissue+label%3A%22help+wanted%22). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
# Development Workflow | ||
|
||
<!-- TOC depthFrom:2 --> | ||
|
||
- [Prerequisites](#prerequisites) | ||
- [Develop locally](#develop-locally) | ||
|
||
<!-- /TOC --> | ||
|
||
|
||
## Prerequisites | ||
|
||
* Install your native toolchain and NativeScript as [described in the docs](https://docs.nativescript.org/start/quick-setup) | ||
|
||
* Review [NativeScript plugins documentation](https://docs.nativescript.org/plugins/plugins) for more details on plugins development | ||
|
||
|
||
## Develop locally | ||
|
||
For local development we recommend using the npm commands provided in the plugin's package.json | ||
|
||
Basically executing a bunch of commands will be enough for you to start making changes to the plugin and see them live synced in the demo. It's up to you to decide which demo to use for development - TypeScript or TypeScript + Angular. | ||
|
||
|
||
To run and develop using TypeScript demo: | ||
```bash | ||
$ cd nativescript-camera/src | ||
$ npm run demo.ios | ||
$ npm run demo.android | ||
``` | ||
|
||
To run and develop using TypeScript + Angular demo: | ||
```bash | ||
$ cd nativescript-camera/src | ||
$ npm run demo.ng.ios | ||
$ npm run demo.ng.android | ||
``` | ||
|
||
After all the changes are done make sure to test them in all the demo apps. | ||
|
||
For details on plugins development workflow, read [NativeScript plugins documentation](https://docs.nativescript.org/plugins/building-plugins#step-2-set-up-a-development-workflow) covering that topic. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Imagepicker -> Camera