-
Notifications
You must be signed in to change notification settings - Fork 145
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
docs: removed developing for contributing
- Loading branch information
Showing
2 changed files
with
97 additions
and
129 deletions.
There are no files selected for viewing
This file contains 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 |
---|---|---|
@@ -1,32 +1,112 @@ | ||
# Development Conventions | ||
# Developing the SDK Locally | ||
*Questions? [Contact us](https://support.branch.io/support/tickets/new)* | ||
|
||
This document outlines our development processes. | ||
1. [Dependencies](#dependencies) | ||
1. [SDK](#sdk) | ||
1. [Develop](#develop) | ||
1. [Validate](#validate) | ||
1. [Test](#test) | ||
1. [Submit](#submit) | ||
|
||
## ES6 | ||
# | ||
|
||
Source code for all .js files in this project is written in es6 and then compiled via babel to es5. Unfortunately, for legacy reasons, the es5 needs to be stored in version control. As such, if you want to change a js file, edit the version in the corresponding .es6 directory, compile it with the `gulp babel` command, and then commit it as well as the resulting .js file. | ||
### Dependencies | ||
|
||
## Pull Requests | ||
> Homebrew | ||
All changes to this project should be made in the form of pull requests against master. Use snake-case prefixed by 'feat/' or 'fix/' for features/fixes. Feel free to use any of the commit types listed [here](https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#type) as well. Note that when pull requests are merged in they will be automatically released to npm by (Travis CI)[https://travis-ci.org/] and (semantic-release)[https://github.com/semantic-release/semantic-release] so make sure to fully test. | ||
```sh | ||
ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"; | ||
brew update; | ||
brew doctor; | ||
export PATH="/usr/local/bin:$PATH"; | ||
``` | ||
|
||
> Node | ||
```sh | ||
brew install node; | ||
``` | ||
|
||
> Gulp | ||
```sh | ||
npm install -g gulp-cli; | ||
``` | ||
|
||
### SDK | ||
|
||
> Local | ||
```sh | ||
git clone git@github.com:BranchMetrics/cordova-ionic-phonegap-branch-deep-linking.git; | ||
cd cordova-ionic-phonegap-branch-deep-linking; | ||
rm -rf node_modules; | ||
npm install --save-dev; | ||
``` | ||
|
||
### Develop | ||
|
||
> Build | ||
## Commit Messages | ||
```sh | ||
gulp prerelease; | ||
``` | ||
|
||
> **[optional]** Update [iOS SDK](https://github.com/BranchMetrics/ios-branch-deep-linking/tags) (will need to update `plugin.xml` dependencies if new iOS files) | ||
Please format your commit messages to match [Angular's Conventions](https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#-git-commit-guidelines). These messages are parsed using semantic release to determine version numbers so it is important that you stick to them exactly. We recommend using the `npm run commit` script installed in this project to draft conformant messages. | ||
```sh | ||
./src/ios/dependencies/update.sh 0.12.20; | ||
``` | ||
|
||
## Linting | ||
### Test | ||
|
||
The style of all js files in this project are strictly checked using [jscs](http://jscs.info/) and [eslint](http://eslint.org/) before any pull request/release is made. You can manually check your local clone using the `gulp lint` command (don't forget to `npm install -g gulp` if you haven't already). Many problems can be automatically fixed by `gulp jscs-fix`. Don't forget to commit before running jscs-fix in case you end up needing to back out changes. | ||
> Modify `BRANCH_KEY` and `URI_SCHEME` and `config.xml` to values in [Branch Dashboard](https://dashboard.branch.io/settings/link) | ||
## Testing | ||
```sh | ||
gulp prerelease; cd testbed; npm install -g cordova; cordova platform remove ios; cordova platform remove android; cordova platform remove browser; cordova platform add ios; cordova platform add android; cordova plugin remove branch-cordova-sdk; cordova plugin add ../ --variable BRANCH_KEY=key_live_icCccJIpd7GlYY5oOmoEtpafuDiuyXhT --variable URI_SCHEME=enefftest; | ||
|
||
See [tests-harness/README.md](https://github.com/BranchMetrics/cordova-ionic-phonegap-branch-deep-linking/blob/master/tests-harness/README.md) for details on how to run this project's test suite. Please run the test suite before merging any branches. | ||
gulp prod; | ||
cd testbed; | ||
npm uninstall mkpath node-version-compare plist xml2js; | ||
rm -rf ../.installed; | ||
rm -rf ./plugins; | ||
rm -rf ./platforms; | ||
cordova platform add ios; | ||
cordova plugin add ../; | ||
cordova build ios --developmentTeam="PW4Q8885U7"; | ||
open -a Xcode platforms/ios/Branch\ Testing.xcworkspace; | ||
|
||
## Updating IOS dependencies | ||
``` | ||
|
||
> Validate all features on both `iOS` and `Android` on `device` only (no `simulator` or `TestFlight`) | ||
This project depends on [ios-branch-deep-linking](https://github.com/BranchMetrics/ios-branch-deep-linking). To upgrade that dependency a [release](https://github.com/BranchMetrics/ios-branch-deep-linking/releases) of it run the following script: | ||
```sh | ||
cordova build ios --developmentTeam="PW4Q8885U7"; open -a Xcode platforms/ios/Branch\ Testing.xcworkspace; | ||
``` | ||
|
||
```shell | ||
# <tag> is the name of the release. e,g, '0.12.5' | ||
$ src/ios/dependencies/update.sh <tag> | ||
```sh | ||
cordova build android; cordova run android; | ||
chrome://inspect/#devices | ||
``` | ||
|
||
> Test harnesses `TODO` | ||
### Submit | ||
|
||
> **Required** [Semantic Release](https://github.com/semantic-release/semantic-release) comments `feat`, `fix`, `docs`, `style`, `refactor`, `perf`, `test`, `chore`, `revert` | ||
> Submit code with a [pull request](https://github.com/BranchMetrics/cordova-ionic-phonegap-branch-deep-linking) | ||
```sh | ||
git checkout -b BRANCH_NAME; | ||
git add FILE_NAME; | ||
npm run commit; | ||
git push origin BRANCH_NAME; | ||
``` | ||
|
||
### Publish | ||
|
||
> Update `CHANGELOG.md` | ||
> Update version within plugin.template.xml | ||
> Run `gulp prerelease` | ||
> Merge Pull Request - code to NPM will happen automatically |
This file was deleted.
Oops, something went wrong.