[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...
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 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...
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 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...
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 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...
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 platforms, another discussion... --- 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...
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-device dependency to test plugin though --- 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...
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 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...
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 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS & Andr...
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 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS Andr...
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 https://github.com/AndyChenW/cordova-plugin-device feature/add_isSimulator_to_iOS_and_android Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cordova-plugin-device/pull/35.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #35 commit 50abd1a7c39ca628d7c63689aa9f93f04ca7ddaf Author: Andy Chen chenyulo...@gmail.com Date: 2015-04-10T06:55:25Z add isSimulator for iOS Android device --- 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS Andr...
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 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS Andr...
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. --- 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS Andr...
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 iOS and emulator for Android and Windows. But we don't want to have two properties or different properties for different platforms. My point was all about being internally consistent: our command to run an app on an emulator/simulator is `cordova emulate`, so to be internally consistent I'd expect the property to be `isEmulator` rather than `isSimulator`. --- 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS Andr...
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 if we get any more opinions. --- 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS Andr...
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 and Windows at least)? --- 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
[GitHub] cordova-plugin-device pull request: add isSimulator for iOS Andr...
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 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
