-
-
Notifications
You must be signed in to change notification settings - Fork 66
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
Full license text in JSON report #222
Comments
Yes. I am always willing to accept PRs!. Can we add this new functionality while keeping the old functionality? |
Hey, sure, we can just add a new config value. A slight problem is that the defaults currently vary between JSON (not included) and HTML (included). Would it be okay with you to change the default of either one of them? Then we could have a config value like |
Yeah. Should be |
@sebastianludwig Do you mind sending a PR for this? |
Hey, I moved projects and am currently not working with Android. If that changes in the future, I'll happily come back to my offer. Sorry for not getting back to you any earlier and for not being able to help :-/ |
Currently only the HTML report contains the full license text. We'd love to use your plugin but we need the full license text in the JSON report. Would you be open to a PR?
The text was updated successfully, but these errors were encountered: