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

gradle_scripts directory purpose? #1

Closed
seato opened this issue Jul 27, 2016 · 4 comments
Closed

gradle_scripts directory purpose? #1

seato opened this issue Jul 27, 2016 · 4 comments

Comments

@seato
Copy link

seato commented Jul 27, 2016

Your last couple of lines in the README have

- Include `app/config` and `app/gradle_scripts`  in your module root
...
- Edit your `app/build.gradle` and include these lines as per your requirements
  - `apply from: 'gradle_scripts/checkstyle.gradle`
  - `apply from: 'gradle_scripts/checkstyle.gradle'`
  - `apply from: 'gradle_scripts/pmd.gradle'`

Are you saying that we need to move our existing gradle scripts into gradle_scripts? Is this directory used anywhere else? I don't see it being referenced so I get the feeling that if our normal build.gradle works with wherever we defined our gradle scripts to be then we can do away with it.

I tried not putting my quality control gradle scripts in that directory and everything seems to work just fine.

@seato seato changed the title gradle_scripts directory purpose gradle_scripts directory purpose? Jul 29, 2016
@DheerendraRathor
Copy link
Owner

Sorry for late reply.

gradle_scripts directory was created to keep all lint related gradle scripts in one folder. You're free to use any convention you want. You can move checkstyle and pmd gradle scripts to your existing directory. Just make sure you're including these scripts (checkstyle, pmd and findbugs) in your main build.gradle.

Hope this helps :-)

@deepakwv
Copy link

deepakwv commented Aug 5, 2016

i have done all setup but when i run arc lint command it show : (Usage Exception: No lint engine is configured for this project. Create an '.arclint' file, or configure an advanced engine with 'lint.engine' in '.arcconfig')

@seato
Copy link
Author

seato commented Aug 5, 2016

@deepakwv This is a separate issue, would you open up a different issue for this conversation?

@seato
Copy link
Author

seato commented Aug 5, 2016

Thanks for the quick response @DheerendraRathor. I just wanted to verify that I wasn't breaking anything by ignoring that part of the README.

@seato seato closed this as completed Aug 5, 2016
This was referenced Aug 17, 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

No branches or pull requests

3 participants