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

fix: update the css loader in order to fix a bug with leading dashes in css classes #847

Merged
merged 1 commit into from
Apr 4, 2019

Conversation

DimitarTachev
Copy link
Contributor

PR Checklist

What is the current behavior?

We were using an old version of the css loader containing the following issue: webpack-contrib/css-loader#578 and causing the below-mentioned issue in NativeScript apps.

What is the new behavior?

The below-mentioned issue is fixed in the latest version of the css-loader.

Related to: #846

@cla-bot cla-bot bot added the cla: yes label Apr 2, 2019
@ghost ghost assigned DimitarTachev Apr 2, 2019
@ghost ghost added new PR review labels Apr 2, 2019
@DimitarTachev
Copy link
Contributor Author

test ng-apps

@DimitarTachev DimitarTachev force-pushed the tachev/fix-css-encoding branch from cd429da to 3fe8c9e Compare April 2, 2019 14:33
@DimitarTachev
Copy link
Contributor Author

test ng-apps

@DimitarTachev
Copy link
Contributor Author

test cli-templates cli-smoke tns_core_modules_unit_tests cuteness uitests modal_navigation animations groceries sdk_ng sdk_js renderer router JavaScriptApp TypeScriptApp AngularApp templates

@DimitarTachev
Copy link
Contributor Author

test cli-templates cli-smoke ng-apps package_version#latest

@vchimev
Copy link
Contributor

vchimev commented Apr 3, 2019

Version 1.0.0 removes minimize option.

@DimitarTachev
Copy link
Contributor Author

run ci

@DimitarTachev DimitarTachev merged commit 7670e33 into master Apr 4, 2019
@ghost ghost removed new PR review labels Apr 4, 2019
@DimitarTachev DimitarTachev deleted the tachev/fix-css-encoding branch April 4, 2019 07:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants