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

README file lists incorrect configuration option #33

Closed
daj opened this issue Nov 16, 2015 · 5 comments
Closed

README file lists incorrect configuration option #33

daj opened this issue Nov 16, 2015 · 5 comments

Comments

@daj
Copy link

daj commented Nov 16, 2015

I tried configuring the dexcount plugin using the options described in the README file:

dexcount {
    includeClasses = false
    includeFieldCount = false
    includeTotalMethodCount = true
    printAsTree = false
    orderByMethodCount = true
    verbose = false
}

Unfortunately my build fails with this error:

* What went wrong:
A problem occurred evaluating project ':app'.
> No such property: includeMethodCount for class: com.getkeepsafe.dexcount.DexMethodCountExtension_Decorated

I tried changing includeTotalMethodCount to includeMethodCount and that didn't help either.

For now I've just commented out that configuration option so my build passes (obviously you can choose to not define the dexcount configuration block and use the defaults - that works fine too).

@benjamin-bader
Copy link
Collaborator

True - this is an option that is not yet in a release build. FYI, you don't need to add a dexcount section to build.gradle at all, unless you need to configure the output. The defaults are pretty sane.

@KishoreBabuIN
Copy link

+1 ... Please resolve..
Is this fixed on the snapshot releases?

@cesards
Copy link

cesards commented Dec 3, 2015

+1 here.

@benjamin-bader
Copy link
Collaborator

I'm on vacation, will fix soon. As noted, you can just remove the option and it will work as expected.

@benjamin-bader
Copy link
Collaborator

This should be resolved with the release of 0.3.1, which includes the documented config option.

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

No branches or pull requests

4 participants