cyclonedx-gradle-plugin icon indicating copy to clipboard operation
cyclonedx-gradle-plugin copied to clipboard

Default config should not include test classpath

Open blommish opened this issue 5 months ago • 1 comments

Connected to

  • https://github.com/CycloneDX/cyclonedx-gradle-plugin/issues/416

https://github.com/CycloneDX/cyclonedx-maven-plugin does not include the test scope as default config.

What has been done

  • To allow setting the list to emptylist, it makes most sense to let set methods overwrite previous values and not use addAll
  • Adding "runtimeClasspath", "compileClasspath", "annotationProcessor" as default configs.
    • These will also include compileOnly, runtimeOnly

For discussion:

Would the classpaths I've added be enough? Or should others also be added? Buildscript, plugin and sourceSet. I'm not totally sure just having the 3 I've added is enough. Or would it be better to just add the test-paths to ignore?

I'll wait to update the readme until I know if this could be used or not.

blommish avatar Sep 06 '24 15:09 blommish