Change CommonOptions to abstract the nativeView instead of platform options.

[nstudio]

Change CommonOptions to abstract the nativeView instead of platform options.

[bradmartin]

Your project shouldn't be analyzing node_modules

https://www.typescriptlang.org/docs/handbook/compiler-options.html search for -skipLibCheck. You can also look at the demo project here for how to set it in the tsconfig.

Reopen if that doesn't solve, but it should.

[nikoTM]

@bradmartin skipLibCheck works, but that is not the default configuration that is generated with nativescript-cli, meaning that if a user creates a new project and adds this plugin the build will fail for no obvious reasons. This solution should probably be mentioned in the readme, but ideally maybe the plugin could use View and extract native view from there, so that there is no dependency on platform declarations?

[bradmartin]

I'm in favor of them changing the templates, I can't see any reason why they aren't already doing that to be honest. It improves tsc performance drastically in most projects with many deps and files to process.

I also lean on the plugin being able to use the platform-declarations that NS provides, so I'll add a note to the README for now and see if they'll update their templates before making changes to the plugin.

Although your idea of grabbing the nativeView is a good idea 👍 so I'll reopen as a future improvement, but that will end up a breaking change if we remove the option, or we have to do a check for view in the common options and then the platform options which adds more code to maintain, so that's a bit discouraging for small OSS projects. Good idea though so thanks for that.