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

Run different tools in parallel #208

Open
szaboa opened this issue Jan 27, 2019 · 6 comments
Open

Run different tools in parallel #208

szaboa opened this issue Jan 27, 2019 · 6 comments

Comments

@szaboa
Copy link

szaboa commented Jan 27, 2019

Hi, I am wondering if it's possible to run these different static code analyzers in parallel? One use case would be to define distinct jobs for let's say FindBug, PMD, Android Lint etc. and execute them in parallel inside a CI workflow.

Does this make sense? Or it would help?
I assume that right know these tools doesn't share any cache or re-use partial result from each other. Am I right?

How could I separate this one gradle task into smaller ones, to be able to run them in parallel?

@stkent
Copy link
Collaborator

stkent commented Jan 28, 2019

As a data point, what sort of size project are you running Gnag on, how many build variants does it have, and how long does gnagCheck take to run (approximately)?

In theory, I think you're right; it should be possible to run these checks in parallel and speed things up a bunch.

In practice, I don't believe the current architecture of this plugin makes that possible. Neither @btkelly or I have a lot of time to devote to Gnag at the moment, but we'd definitely be happy to consider a proposal in the form of a PR if you want to take a crack at the necessary changes. Let us know if that's something that interests you and we can support where needed.

@szaboa
Copy link
Author

szaboa commented Jan 28, 2019

Well this was just a theoretical question, right now I am just experimenting with different CI workflows on Android so I can't say that the current task is slow, but I am pretty sure I will use this this plugin in production apps as well. I will look into this and do a PR if I succeed :)

@stkent
Copy link
Collaborator

stkent commented Jan 29, 2019

Awesome, glad to hear it! :)

@szaboa
Copy link
Author

szaboa commented Feb 21, 2019

Pull request is done, waiting some feedback there :)

@stkent
Copy link
Collaborator

stkent commented Feb 21, 2019

Poke @btkelly :)

@szaboa
Copy link
Author

szaboa commented Mar 6, 2019

@btkelly, @stkent ping :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants