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

fix: exclude impl core modules classes from mangling #173

Merged
merged 1 commit into from
Jun 6, 2017

Conversation

PeterStaev
Copy link
Contributor

The classes that are excluded for android are incorrect. Not the alias but the actual class that is decorated with @Interfaces() decorated should be excluded from the mangling. This is because the runtime creates the java helper classes based on the names of the classes that are decorated. So excluding the aliases has no effect.

@ns-bot
Copy link

ns-bot commented Jun 2, 2017

Can one of the admins verify this patch?

4 similar comments
@ns-bot
Copy link

ns-bot commented Jun 2, 2017

Can one of the admins verify this patch?

@ns-bot
Copy link

ns-bot commented Jun 2, 2017

Can one of the admins verify this patch?

@ns-bot
Copy link

ns-bot commented Jun 2, 2017

Can one of the admins verify this patch?

@ns-bot
Copy link

ns-bot commented Jun 2, 2017

Can one of the admins verify this patch?

@sis0k0
Copy link
Contributor

sis0k0 commented Jun 5, 2017

run ci

1 similar comment
@sis0k0
Copy link
Contributor

sis0k0 commented Jun 5, 2017

run ci

@sis0k0 sis0k0 changed the title Fix android excludes fix: exclude impl core modules classes from mangling Jun 6, 2017
@sis0k0 sis0k0 merged commit 53d7538 into NativeScript:master Jun 6, 2017
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.

3 participants