Skip to content

fix(nsconfig):app resources not filtered on prepare #3429

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 8, 2018

Conversation

KristianDD
Copy link
Contributor

@KristianDD KristianDD commented Mar 8, 2018

Fixes #3428

@petekanev
Copy link
Contributor

Fixes #3428

@petekanev petekanev self-requested a review March 8, 2018 09:05
@@ -17,7 +17,7 @@ export class AppFilesUpdater {

// exclude the app_resources directory from being enumerated
// for copying if it is present in the application sources dir
const appResourcesPathNormalized = path.normalize(projectData.appResourcesDirectoryPath + "\\");
const appResourcesPathNormalized = path.normalize(projectData.appResourcesDirectoryPath + path.sep);
Copy link
Contributor

Choose a reason for hiding this comment

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

why do you need / at the end ?

Copy link
Contributor

@petekanev petekanev Mar 8, 2018

Choose a reason for hiding this comment

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

@rosen-vladimirov
So that there are no false-positives when there is a directory which contains a similar name.

For example, if your resources remain as App_Resources, but you've got another App_Resources123 directory, you don't want that filtered out. Similarly if you rename the resources directory to something more generic like resources, and if you have another resources_home_page, the latter should not be filtered out of the application files.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Pip3r4o added it so we can filter more accurately. For example if you have Resources folder named "native" and folder named "native_classes" we want to filter only files inside "native".

@petekanev
Copy link
Contributor

run ci

1 similar comment
@KristianDD
Copy link
Contributor Author

run ci

@petekanev petekanev changed the title fix(nsconfig):app reources not filtered on prepare fix(nsconfig):app resources not filtered on prepare Mar 8, 2018
@petekanev petekanev merged commit 7bb2248 into master Mar 8, 2018
@petekanev petekanev deleted the kddimitrov/fix-filter-resources-prepare branch March 8, 2018 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants