-
-
Notifications
You must be signed in to change notification settings - Fork 98
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
Handle missing build.gradle file #86
Comments
Having a Gradle module without a build.gradle file smells a bit iffy to me. |
Yeah, I don't really know what to say to that, except that Gradle supports submodules without a build file and no build file smells better to me than an empty build file. Since this is becoming a war of the noses and quite subjective feel free to close this. I'll just add an empty build file when I generate dependency graphs, hell, I might event keep it under source control, since I like this plugin so much :). |
What does the Gradle API say though? Is it really allowed to be null? If it is then we should honor that. |
@bjorndarri want to create a PR that handles the case where a build.gradle file is missing? |
I'm afraid I have practically zero experience with Gradle plugin development and Kotlin, so I don't think I'm the one for the job. I may look into this at some point though, since I'm interested in both Gradle and Kotlin, but I wouldn't bet on me fixing this issue any day soon. |
Do you have a sample repo how I can set up a Gradle module without a build.gradle file? |
I've created a minimal multi module example where the api module does not have a build.gradle file. The generateDependencyGraph task fails when run for the api module. |
Thanks. I'll have a look when I have some spare time! |
I have a submodule without a build.gradle file, since it has no configuration or dependencies other than what is specified in the root module build file.
This results in the following error:
I solved this by creating an empty build.gradle file, but keeping an empty build.gradle file smells a bit iffy.
The text was updated successfully, but these errors were encountered: