-
Notifications
You must be signed in to change notification settings - Fork 0
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
Support subprojects in export dependencies to TeamCity #53
Conversation
fishinitself
commented
Jun 20, 2024
- Capability to automatically export dependencies to TeamCity for each Gradle subproject individually.
- Capability to explicitly declare dependencies through the plugin DSL and export them to TeamCity for each Gradle subproject individually.
...ces/teamcity-dependencies-registration/sub-projects/gradle/wrapper/gradle-wrapper.properties
Show resolved
Hide resolved
ft/src/test/resources/teamcity-dependencies-registration/template-all-deps-root_prj.properties
Outdated
Show resolved
Hide resolved
ft/src/test/resources/teamcity-dependencies-registration/template-all-deps-root_prj.properties
Outdated
Show resolved
Hide resolved
ft/src/test/resources/teamcity-dependencies-registration/template-all-deps-root_prj.properties
Show resolved
Hide resolved
-PbuildVersion=1.0-SNAPSHOT \ | ||
-Pversion=1.0-SNAPSHOT \ | ||
-Dorg.gradle.internal.publish.checksums.insecure=true \ | ||
-Plicense-management.version=0.1.89 \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Не нужно
ft/src/test/resources/teamcity-dependencies-registration/template-all-deps-root_prj.properties
Show resolved
Hide resolved
-Dorg.gradle.internal.publish.checksums.insecure=true \ | ||
-Plicense-management.version=0.1.89 \ | ||
-Plicense-maven-plugin.version=1.14.230 \ | ||
-Plicense.skip=false \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Не нужно
@@ -0,0 +1,19 @@ | |||
command-and-arguments=-Poctopus-release-management.version=__RELEASE_MANAGEMENT_VERSION__ \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Давай уберем legacy как в прошлом примере
@@ -0,0 +1,18 @@ | |||
command-and-arguments=-Poctopus-release-management.version=__RELEASE_MANAGEMENT_VERSION__ \ | |||
-PbuildVersion=1.0-SNAPSHOT \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Убрать legacy
@@ -0,0 +1,17 @@ | |||
command-and-arguments=-Poctopus-release-management.version=__RELEASE_MANAGEMENT_VERSION__ \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Убрать legacy
This reverts commit b59279b.
Arguments.of("multi-module-with-root-publish-4.10.3", listOf("multi-module-with-root-publish-4.10.3", "module-5")), | ||
Arguments.of("legacy-staging-plugin", listOf("deployer-dsl-core", "deployer-dsl-file")) | ||
Arguments.of("multi-module-with-root-publish-4.10.3", listOf("multi-module-with-root-publish-4.10.3", "module-5")) | ||
// Arguments.of("legacy-staging-plugin", listOf("deployer-dsl-core", "deployer-dsl-file")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gradle-staging-plugin не используется, его можно удалить (м.б. отдельным PR?)
Но возможно в этом тесте есть сценарий, который не покрывается другими тестами? т.е. просто заменив gradle-staging-plugin -> rm-gradle-plugin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Т.е. тест предлагаю не выключать, а упростить\заменить на проверку того, что конструкции вида escrowBuild, nexusStaging по прежнему поддержаны. (других тестов на это я не вижу)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done