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

Enable dexcount plugin #185

Merged

Conversation

serj-lotutovici
Copy link
Contributor

Count methods on each build with -PwithDexcount

@artem-zinnatullin
Copy link
Owner

Process 'Gradle Test Executor 2' finished with non-zero exit value 137

Not enough memory to do the build.

I have a branch where I divide execution of build steps to reduce memory usage, u can take a look at it here 9322fd1

@codecov-io
Copy link

codecov-io commented Jun 23, 2016

Current coverage is 78.01%

Merging #185 into master will not change coverage

@@             master       #185   diff @@
==========================================
  Files            38         38          
  Lines           555        555          
  Methods           0          0          
  Messages          0          0          
  Branches         47         47          
==========================================
  Hits            433        433          
  Misses           92         92          
  Partials         30         30          

Powered by Codecov. Last updated by d33d7a4...bbb1232

apply plugin: 'com.getkeepsafe.dexcount'

dexcount {
format = "list"

Choose a reason for hiding this comment

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

single quotes

@artem-zinnatullin
Copy link
Owner

👍

@artem-zinnatullin
Copy link
Owner

@vanniktech, @lenguyenthanh PTAL

Count methods on each build with -PwithDexcount
@serj-lotutovici
Copy link
Contributor Author

Btw, the current method count is:

Total methods in app-1.0-1-debug.apk: 49745 (75.91% used)
Total fields in app-1.0-1-debug.apk:  25691 (39.20% used)
Methods remaining in app-1.0-1-debug.apk: 15790
Fields remaining in app-1.0-1-debug.apk:  39844

I'm still looking for a way to publish the values on github pr, using travis or any other ci. At my company we are using Jenkins, and there are a few plugins for that.

@vanniktech
Copy link
Contributor

👍

@lenguyenthanh
Copy link
Contributor

👍 this plugin is really cool.

@artem-zinnatullin artem-zinnatullin merged commit af459fa into artem-zinnatullin:master Jun 24, 2016
@artem-zinnatullin
Copy link
Owner

Thanks, @serj-lotutovici!

@serj-lotutovici serj-lotutovici deleted the sl/add_dexcount branch June 24, 2016 08:21
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.

5 participants