Skip to content
This repository has been archived by the owner on Jan 18, 2021. It is now read-only.

ComparePublications working for Gradle plugins #419 #461

Merged
merged 1 commit into from
Oct 2, 2017
Merged

Conversation

wwilk
Copy link
Contributor

@wwilk wwilk commented Oct 1, 2017

I tested it locally. It seems that there is always a difference in poms but since this is not a critical situation, we can test if the same problem occurs on Travis.

Copy link
Member

@mockitoguy mockitoguy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice! Merging!

@@ -8,6 +8,9 @@ apply plugin: 'com.gradle.plugin-publish'

apply from: "$rootDir/gradle/upgrade-downstream.gradle"

//TODO: WW uncomment when GradlePluginArtifactUrlResolver is released
//apply plugin: 'org.shipkit.compare-publications'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why don't we fold it under 'org.shipkit.gradle-plugin'? It's already very opinionated by design.


public class DefaultArtifactUrlResolverFactory {

public DefaultArtifactUrlResolver getDefaultResolver(Project project, String artifactBaseName, String previousVersion) {
if (project.getPlugins().hasPlugin(ShipkitBintrayPlugin.class)) {
return new BintrayDefaultArtifactUrlResolver(project, artifactBaseName, previousVersion);
} else if (project.getPlugins().hasPlugin(GradlePortalPublishPlugin.class)) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really nice idea, thank you!

@mockitoguy mockitoguy merged commit 768ab04 into master Oct 2, 2017
@wwilk wwilk deleted the ww branch October 2, 2017 19:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants