[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-10-24 Thread stevengill
Github user stevengill commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-150849472 Haha we were just chatting about this yesterday. Makes most sense to add the dependency to tests/plugin.xml --- If your project is set up for it, you

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-10-24 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-150835775 Love it! Shockingly, @stevengill and I had this idea also. --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-10-24 Thread alsorokin
Github user alsorokin commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-150781035 @purplecabbage what if we add device dependency not to accelerometer plugin itself but to tests/plugin.xml ? --- If your project is set up for it, you

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-10-23 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-150621997 Yes. Except it is only useful for plugins that depends on device. Accelerometer should not depend on device. I would like to move this directly into

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-10-23 Thread alsorokin
Github user alsorokin commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-150563793 @nikhilkh this is essentially what I needed so I will use this to pend unsupported tests on iOS simulator I guess this will add a cordova-plugin-devi

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-10-23 Thread nikhilkh
Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-150561368 @alsorokin Isn't this what you needed for disabling failing tests on iOS simulator? --- If your project is set up for it, you can reply to this email an

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-10-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-device/pull/35 --- 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 f

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-10-22 Thread TimBarham
Github user TimBarham commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-150308646 Works for me. --- 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 h

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-10-22 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-150307876 Created an issue for this here: CB-9847 Moving forward with `isVirtual` --- If your project is set up for it, you can reply to this email and ha

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-07-08 Thread AndyChenW
Github user AndyChenW commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-119817738 device.isVirtual +1 --- 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 doe

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-07-08 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-119817528 Ah, gotcha. I actually think I like isVirtual better as it sidesteps all the sim/emu discussions. I'll let this pr sit for a bit and see i

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-07-08 Thread TimBarham
Github user TimBarham commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-119813107 Heh, I wasn't wanting to start a semantics argument, just after consistency. My understanding is that the usual terminology is "simulator" for iO

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-07-08 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-119811586 2 other 'offers' device.isVirtual Or the opposite device.isDevice --- If your project is set up for it, you can reply to this em

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-07-08 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-119809686 Attempting to avoid a semantics argument, I chose to overlook this. do we really need to add 2 properties? iOS simulator IS a simulator. --- I

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-07-08 Thread TimBarham
Github user TimBarham commented on the pull request: https://github.com/apache/cordova-plugin-device/pull/35#issuecomment-119804386 Should this be "Emulator" rather than "Simulator" (that's the terminology used elsewhere in Cordova, isn't it? And the typical terminology for Android a

[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...

2015-07-08 Thread AndyChenW
GitHub user AndyChenW opened a pull request: https://github.com/apache/cordova-plugin-device/pull/35 add isSimulator for iOS & Android device @purplecabbage change pr and property name to isSimulator. You can merge this pull request into a Git repository by running: $ git pull