You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, in order to test changes to the test-collector-plugin, the plugin must be published to the local maven repository using ./gradlew publishToMavenLocal or reference the latest published snapshot. This is very inefficient, as the command must be performed in order to test even a minor modification. Since the CI platforms have no maven repository server where we may publish the SDK prior to building the project, the CI workflows will fail if the added dependency to access the plugin published in maven local is not commented out or deleted.
Also, unlike the instrumented-test-collector library, where the library module can be added as a project dependency, i.e. implementation(project("collector:instrumented-test-collector")), the plugin module cannot be directly referenced. Hence, the implementation of the plugin should be adapted in order to make it easily testable without publishing it locally.
💬 Context
Currently, in order to test changes to the
test-collector-plugin
, the plugin must be published to the local maven repository using./gradlew publishToMavenLocal
or reference the latest published snapshot. This is very inefficient, as the command must be performed in order to test even a minor modification. Since the CI platforms have no maven repository server where we may publish the SDK prior to building the project, the CI workflows will fail if the added dependency to access the plugin published in maven local is not commented out or deleted.Also, unlike the instrumented-test-collector library, where the library module can be added as a project dependency, i.e.
implementation(project("collector:instrumented-test-collector"))
, the plugin module cannot be directly referenced. Hence, the implementation of the plugin should be adapted in order to make it easily testable without publishing it locally.📝 Notes
🤝 Related
The text was updated successfully, but these errors were encountered: