Skip to content
This repository has been archived by the owner on Aug 7, 2021. It is now read-only.

refactor(angular): provide mainPath instead of entryModule to ng compiler plugin #633

Merged
merged 1 commit into from
Aug 14, 2018

Conversation

sis0k0
Copy link
Contributor

@sis0k0 sis0k0 commented Aug 13, 2018

The entryModule path can differ depending on what kind of project you're
using - mobile or web+mobile. While the best solution is to get the
mainPath from angular.json, not all {N} projects have that file. We'll
stick to the standard for web and web+mobile location - appPath/main.ts.

Note - We'll have to update {N} schematics to reflect this change.

compiler plugin

The entryModule path can differ depending on what kind of project you're
using - mobile or web+mobile. While the best solution is to get the
mainPath from angular.json, not all {N} projects have that file. We'll
stick to the standard for web and web+mobile location - `appPath/main.ts`.
Copy link

@sebawita sebawita left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this change, as it simplifies the configuration and it makes it easier to change the EntryModule.
We need to make sure to sync it with @nativescript/schematics release

@sis0k0 sis0k0 merged commit 42b0c0b into master Aug 14, 2018
@sis0k0 sis0k0 deleted the sis0k0/mainpath branch August 14, 2018 12:40
manoldonev pushed a commit to NativeScript/nativescript-app-templates that referenced this pull request Jan 9, 2019
…#86)

The following changes are in this PR:
- the app folder is renamed to src as is in the web Angular projects.
- @nativescript/schematics is a devDependency and angular.json file is added. This allows the developers to start using the Angular CLI immediately after creating a project.
- App_Resources are now at root level.
- nsconfing.json file is added.

depends on NativeScript/nativescript-dev-webpack#633 and NativeScript/nativescript-dev-typescript#61
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants