Skip to content
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

feat(css): add attribute scoped css without global refresh (#7907) #7947

Merged
merged 1 commit into from Oct 14, 2019

Conversation

VladimirAmiorkov
Copy link
Contributor

This is a patch PR for #7907

@VladimirAmiorkov VladimirAmiorkov self-assigned this Oct 11, 2019
@cla-bot cla-bot bot added the cla: yes label Oct 11, 2019
@MartoYankov
Copy link
Contributor

This PR is basically adding a new feature. I'm not sure this is good for a patch. Also, I've communicated with the guy who initially reported the issue and helped me isolate it that it will come out with {N} 6.2 at the end of the month.

@manoldonev
Copy link
Contributor

manoldonev commented Oct 14, 2019

@MartoYankov Generally I agree with you but as this change is triggered by a nativescript-angular fix, we need to release it with a patch in tns-core-modules as with 6.2.0 we'll be releasing a patch version of nativescript-angular (nativescript-angular versioning is tied to angular major and minor releases) that will rely on this fix and otherwise it will break all users (e.g. if you have a dependency like "nativescript-angular": "~8.2.0").

@manoldonev manoldonev merged commit b377eb6 into release Oct 14, 2019
@manoldonev manoldonev deleted the amiorkov/scoped-css-patch branch October 14, 2019 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants