Github user omefire commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62777073
--- Diff: src/telemetry.js ---
@@ -0,0 +1,132 @@
+/**
+Licensed to the Apache Software Foundation (ASF) under one
+or more
Github user omefire commented on the pull request:
https://github.com/apache/cordova-docs/pull/599#issuecomment-218331338
Thanks, comments addressed.
---
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
Github user omefire commented on a diff in the pull request:
https://github.com/apache/cordova-docs/pull/599#discussion_r62776600
--- Diff: www/privacy/index.md ---
@@ -0,0 +1,22 @@
+---
+layout: page
+title: Privacy Policy
+---
+
+# Privacy Policy
+
I expect it to be merged in by tomorrow EOD or day Thursday morning.
If it's still not merged in by then, I wouldn't want to hold the release any
longer.
Thanks,
Omar Mefire
-Original Message-
From: Steven Gill [mailto:stevengil...@gmail.com]
Sent: Tuesday, May 10, 2016 2:39 PM
To:
Github user purplecabbage commented on a diff in the pull request:
https://github.com/apache/cordova-ios/pull/187#discussion_r62772944
--- Diff: CordovaLib/Classes/Public/CDVPlugin.m ---
@@ -25,11 +25,11 @@ Licensed to the Apache Software Foundation (ASF) under
one
Github user jasongin commented on the pull request:
https://github.com/apache/cordova-windows/pull/174#issuecomment-218324502
Oh, yeah I'll update those tests.
---
In reply to:
[218305527](https://github.com/apache/cordova-windows/pull/174#issuecomment-218305527)
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-ios/pull/189
---
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
Github user shazron commented on the pull request:
https://github.com/apache/cordova-ios/pull/207#issuecomment-218323850
Any update on this? If not I will probably close this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user shazron commented on the pull request:
https://github.com/apache/cordova-ios/pull/187#issuecomment-218323149
I understand why this change should be made. However, since it is part of
the public API (it is part of CDVPlugin), any changes to this will have to go
in a major
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-ios/pull/197
---
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
Github user shazron commented on the pull request:
https://github.com/apache/cordova-ios/pull/197#issuecomment-218318589
Successfully manual test by creating a new project and adding the platform
by specifying this repo with the changes and:
```
cordova emulate ios
cordova
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-media-capture/pull/51
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-media-capture/pull/60
---
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
Github user purplecabbage commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62768428
--- Diff: src/telemetry.js ---
@@ -0,0 +1,132 @@
+/**
+Licensed to the Apache Software Foundation (ASF) under one
+or more
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62767415
--- Diff: src/telemetry.js ---
@@ -0,0 +1,132 @@
+/**
+Licensed to the Apache Software Foundation (ASF) under one
+or more
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62766681
--- Diff: doc/cordova.txt ---
@@ -23,6 +23,12 @@ Project Commands
serve .. Run project with a local webserver
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-docs/pull/599#discussion_r62766491
--- Diff: www/privacy/index.md ---
@@ -0,0 +1,22 @@
+---
+layout: page
+title: Privacy Policy
+---
+
+# Privacy Policy
+
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-docs/pull/599#discussion_r62766349
--- Diff: www/privacy/index.md ---
@@ -0,0 +1,22 @@
+---
+layout: page
+title: Privacy Policy
+---
+
+# Privacy Policy
Github user purplecabbage commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62765915
--- Diff: src/telemetry.js ---
@@ -0,0 +1,132 @@
+/**
+Licensed to the Apache Software Foundation (ASF) under one
+or more
Github user axemclion commented on a diff in the pull request:
https://github.com/apache/cordova-docs/pull/599#discussion_r62764381
--- Diff: www/privacy/index.md ---
@@ -0,0 +1,22 @@
+---
+layout: page
+title: Privacy Policy
+---
+
+# Privacy Policy
Github user axemclion commented on a diff in the pull request:
https://github.com/apache/cordova-docs/pull/599#discussion_r62764282
--- Diff: www/privacy/index.md ---
@@ -0,0 +1,22 @@
+---
+layout: page
+title: Privacy Policy
+---
+
+# Privacy Policy
+
Github user axemclion commented on a diff in the pull request:
https://github.com/apache/cordova-docs/pull/599#discussion_r62764339
--- Diff: www/privacy/index.md ---
@@ -0,0 +1,22 @@
+---
+layout: page
+title: Privacy Policy
+---
+
+# Privacy Policy
+
Github user nikhilkh commented on the pull request:
https://github.com/apache/cordova-windows/pull/174#issuecomment-218305527
Looks like there are some unit test failures
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-docs/pull/501
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-docs/pull/515
---
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
GitHub user nikhilkh opened a pull request:
https://github.com/apache/cordova-docs/pull/600
Update SQLite plugin details
@brodybits to help review.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nikhilkh/cordova-docs patch-6
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-docs/pull/590
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-docs/pull/564
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-docs/pull/589
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-docs/pull/566
---
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
Github user omefire commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62759764
--- Diff: src/telemetry.js ---
@@ -0,0 +1,132 @@
+/**
+Licensed to the Apache Software Foundation (ASF) under one
+or more
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-docs/pull/579
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-docs/pull/587
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-docs/pull/594
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-docs/pull/597
---
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
Github user omefire commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62758588
--- Diff: spec/cli.spec.js ---
@@ -50,19 +56,34 @@ describe("cordova cli", function () {
beforeEach(function () {
});
Github user omefire commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62758273
--- Diff: doc/telemetry.txt ---
@@ -0,0 +1,24 @@
+Synopsis
+
+cordova-cli telemetry [STATE]
+
+ STATE: on|off
+
When do you expect to merge it in?
On Tue, May 10, 2016 at 2:19 PM, Mefire O. wrote:
> I'd like to get the telemetry feature out with this release.
>
> Thanks,
> Omar Mefire
>
> -Original Message-
> From: Vladimir Kotikov (Akvelon) [mailto:v-vlk...@microsoft.com]
Github user omefire commented on the pull request:
https://github.com/apache/cordova-cli/pull/247#issuecomment-218299896
Related: https://github.com/apache/cordova-docs/pull/599
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user omefire commented on the pull request:
https://github.com/apache/cordova-docs/pull/599#issuecomment-218299426
What it looks like: http://imgur.com/FSDAifF
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user omefire commented on the pull request:
https://github.com/apache/cordova-docs/pull/599#issuecomment-218298615
http://imgur.com/FSDAifF
---
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
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-docs/pull/599#discussion_r62755937
--- Diff: www/privacy/index.md ---
@@ -0,0 +1,21 @@
+---
+layout: page
+title: Privacy Policy
+---
+
+# Privacy Policy
+
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-docs/pull/599#discussion_r62755904
--- Diff: www/privacy/index.md ---
@@ -0,0 +1,21 @@
+---
+layout: page
+title: Privacy Policy
+---
+
+# Privacy Policy
+
Github user omefire commented on the pull request:
https://github.com/apache/cordova-docs/pull/599#issuecomment-218296850
@axemclion, @nikhilkh to review
---
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
GitHub user omefire opened a pull request:
https://github.com/apache/cordova-docs/pull/599
Telemetry: Privacy Policy
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/omefire/cordova-docs TelemetryPrivacyPolicy
Alternatively you
I'd like to get the telemetry feature out with this release.
Thanks,
Omar Mefire
-Original Message-
From: Vladimir Kotikov (Akvelon) [mailto:v-vlk...@microsoft.com]
Sent: Tuesday, May 10, 2016 12:47 AM
To: dev@cordova.apache.org
Subject: RE: [DISCUSS] Cordova tools release
+1
Thanks
Github user purplecabbage commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/213#issuecomment-218282825
great points @tony-- ! +1 to all of 'em
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user tony-- commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/213#issuecomment-218277348
There are a couple places that contributor workflow is documented:
The wiki has a [contributor workflow
Github user purplecabbage commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/213#issuecomment-218265034
+1 to the PR title suggestion
The icla is NOT actually required for smaller changes, and we should only
be enforcing this for large/deep
Github user riknoll commented on a diff in the pull request:
https://github.com/apache/cordova-paramedic/pull/5#discussion_r62734243
--- Diff: lib/utils/utilities.js ---
@@ -50,10 +50,14 @@ function getSimId() {
if (findSimResult.code > 0) {
It has been a while since all of us met, so I was wondering if it would help to
do a Cordova Monthly hangout ? I also understand that we have Phonegap day EU
this month, and maybe we could use that timeframe to get more folks involved in
the hangout ? For the rest of us who could not attend
Github user riknoll commented on a diff in the pull request:
https://github.com/apache/cordova-paramedic/pull/5#discussion_r62733686
--- Diff: lib/paramedic.js ---
@@ -175,39 +178,58 @@ ParamedicRunner.prototype.runTests = function() {
reject(new Error('device is
Github user riknoll commented on a diff in the pull request:
https://github.com/apache/cordova-paramedic/pull/5#discussion_r62733344
--- Diff: lib/ParamedicTargetChooser.js ---
@@ -0,0 +1,127 @@
+var Q = require('q');
+var shelljs = require('shelljs');
+var path
Github user riknoll commented on a diff in the pull request:
https://github.com/apache/cordova-paramedic/pull/5#discussion_r62733203
--- Diff: lib/ParamedicTargetChooser.js ---
@@ -0,0 +1,127 @@
+var Q = require('q');
+var shelljs = require('shelljs');
+var path
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-paramedic/pull/5#discussion_r62730561
--- Diff: lib/ParamedicAppUninstall.js ---
@@ -0,0 +1,66 @@
+var shelljs = require('shelljs');
+var path= require('path');
+var fs
Github user riknoll commented on the pull request:
https://github.com/apache/cordova-plugin-camera/pull/213#issuecomment-218253991
Do we need to provide a link for CCLA info just in case? Can we also
suggest a format for PR titles? Something like:
```
[CB-12345]: Short
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-paramedic/pull/5#discussion_r62729774
--- Diff: lib/ParamedicAppUninstall.js ---
@@ -0,0 +1,66 @@
+var shelljs = require('shelljs');
+var path= require('path');
+var fs
I have submitted a PR to add a pull request to cordova-plugin-camera:
https://github.com/apache/cordova-plugin-camera/pull/213
Please review and provide feedback. I intend to add this to all other repos.
-Nikhil
-Original Message-
From: Dmitry Blotsky [mailto:dblot...@microsoft.com]
GitHub user nikhilkh opened a pull request:
https://github.com/apache/cordova-plugin-camera/pull/213
Add pull request template
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/MSOpenTech/cordova-plugin-camera pr-template
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-medic/pull/102
---
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
Github user riknoll commented on the pull request:
https://github.com/apache/cordova-medic/pull/102#issuecomment-218225885
@stevengill no worries! Thanks for taking a look!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62710076
--- Diff: src/cli.js ---
@@ -235,41 +317,58 @@ function cli(inputArgs) {
throw new CordovaError(msg);
}
-
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62709210
--- Diff: src/telemetry.js ---
@@ -0,0 +1,132 @@
+/**
+Licensed to the Apache Software Foundation (ASF) under one
+or more
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62708834
--- Diff: src/telemetry.js ---
@@ -0,0 +1,132 @@
+/**
+Licensed to the Apache Software Foundation (ASF) under one
+or more
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62708649
--- Diff: src/cli.js ---
@@ -235,41 +317,58 @@ function cli(inputArgs) {
throw new CordovaError(msg);
}
-
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62708496
--- Diff: src/cli.js ---
@@ -235,41 +317,58 @@ function cli(inputArgs) {
throw new CordovaError(msg);
}
-
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62708439
--- Diff: src/cli.js ---
@@ -235,41 +317,58 @@ function cli(inputArgs) {
throw new CordovaError(msg);
}
-
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62707187
--- Diff: src/telemetry.js ---
@@ -0,0 +1,132 @@
+/**
+Licensed to the Apache Software Foundation (ASF) under one
+or more
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62705738
--- Diff: spec/cli.spec.js ---
@@ -50,19 +56,34 @@ describe("cordova cli", function () {
beforeEach(function () {
});
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62705505
--- Diff: doc/telemetry.txt ---
@@ -0,0 +1,24 @@
+Synopsis
+
+cordova-cli telemetry [STATE]
+
+ STATE: on|off
+
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62705209
--- Diff: doc/telemetry.txt ---
@@ -0,0 +1,24 @@
+Synopsis
+
+cordova-cli telemetry [STATE]
+
+ STATE: on|off
+
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62704961
--- Diff: doc/telemetry.txt ---
@@ -0,0 +1,24 @@
+Synopsis
+
+cordova-cli telemetry [STATE]
+
+ STATE: on|off
+
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62704887
--- Diff: doc/cordova.txt ---
@@ -23,6 +23,12 @@ Project Commands
serve .. Run project with a local webserver
Github user nikhilkh commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/247#discussion_r62704632
--- Diff: doc/cordova.txt ---
@@ -43,4 +49,7 @@ Examples
cordova-cli requirements android
cordova-cli build android --verbose
The proposal is to create a new core plugin that enables the user to
dynamically control the orientation of their app.
This plugin would be based on the W3C screen-orientation spec [1].
This was originally proposed via CB-9528 [2] (which currently has 10 votes and
16 watchers, indicating strong
Small update.
Looks like failing jshint in https://github.com/apache/cordova-lib/pull/437 is
another Node 6.x (In particular npm@3.x) issue. Corresponding bug is filed
here: https://github.com/npm/npm/issues/10343.
Hopefully this doesn't affect regular users, as the problem only can be seen
Github user codecov-io commented on the pull request:
https://github.com/apache/cordova-lib/pull/437#issuecomment-218113832
## [Current coverage][cc-pull] is **80.59%**
> Merging [#437][cc-pull] into [master][cc-base-branch] will not change
coverage
```diff
@@
+1
Thanks for handling cordova-common release - there is probably not much sense
in releasing it separately (as I was planning initially) when other components
also need to be released.
-
Best regards, Vladimir
-Original Message-
From: julio cesar sanchez
Github user CSharpFan commented on the pull request:
https://github.com/apache/cordova-windows/pull/175#issuecomment-218070737
`npm install lodash --save`
---
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
Github user stevengill commented on the pull request:
https://github.com/apache/cordova-medic/pull/102#issuecomment-218068505
LGTM too! Sorry I forgot about medic :(
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user robertoandrade commented on a diff in the pull request:
https://github.com/apache/cordova-lib/pull/399#discussion_r62616494
--- Diff: cordova-lib/package.json ---
@@ -40,7 +40,7 @@
"underscore": "1.7.0",
"unorm": "1.3.3",
Github user robertoandrade commented on a diff in the pull request:
https://github.com/apache/cordova-lib/pull/399#discussion_r62616280
--- Diff: cordova-lib/package.json ---
@@ -40,7 +40,7 @@
"underscore": "1.7.0",
"unorm": "1.3.3",
82 matches
Mail list logo