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

Added support to instant run #559

Closed
wants to merge 2 commits into from
Closed

Added support to instant run #559

wants to merge 2 commits into from

Conversation

firaskafri
Copy link
Contributor

This was a headache to all of us using the new Android Studio with Instant Run feature

Added a few statements to read dex names and paths in the newly created instant-run directory when using the Instant Run feature inside the already existing method getSourcePaths. which is called by also the already existing method getAllClasses

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @bendaniel10 and @nurolopher to be potential reviewers

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 67.433% when pulling 4b75e4d on firasalkafri:master into 2358dae on satyan:master.

@sibelius
Copy link
Contributor

sibelius commented Apr 3, 2016

@firasalkafri I will take a look tomorrow (Monday), thanks

@JonatanSalas
Copy link
Collaborator

@firasalkafri could you solve the conflict? I will take a look to your code if solved this.

@firaskafri
Copy link
Contributor Author

@JonatanSalas I don't see any conflicts :/

@firaskafri
Copy link
Contributor Author

@JonatanSalas now I see it, resolved.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 69.685% when pulling 9108e79 on firasalkafri:master into fbd7475 on satyan:master.

@sibelius
Copy link
Contributor

sibelius commented Apr 5, 2016

@firasalkafri could you rebase your commits?

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 69.685% when pulling e974625 on firasalkafri:master into fbd7475 on satyan:master.

mitchyboy9 and others added 2 commits April 5, 2016 11:11
…rName can also throw LinkageError and ExceptionInInitializerError.

If we silently ignore a ClassNotFoundException,  I don't see why we wouldn't ignore the others.
A NoClassDefFoundError coming out of this method was causing my application to crash, but it was actually fine that the class couldn't be loaded.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 69.747% when pulling 77828dc on firasalkafri:master into fbd7475 on satyan:master.

@firaskafri firaskafri closed this Apr 5, 2016
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.

None yet

6 participants