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

Add documentation how to ship with maven-publish plugin fixes #768#832

Merged
mstachniuk merged 2 commits into
masterfrom
ms-maven-publish
Aug 23, 2019
Merged

Add documentation how to ship with maven-publish plugin fixes #768#832
mstachniuk merged 2 commits into
masterfrom
ms-maven-publish

Conversation

@mstachniuk

Copy link
Copy Markdown
Contributor

No description provided.

@mstachniuk mstachniuk requested review from epeee and mockitoguy July 12, 2019 15:40
@codecov

codecov Bot commented Jul 12, 2019

Copy link
Copy Markdown

Codecov Report

Merging #832 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #832   +/-   ##
=========================================
  Coverage     76.78%   76.78%           
  Complexity     1227     1227           
=========================================
  Files           219      219           
  Lines          4454     4454           
  Branches        333      333           
=========================================
  Hits           3420     3420           
  Misses          922      922           
  Partials        112      112

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a85921...3f11050. Read the comment docs.


#### Nitty gritty

Applying "[org.shipkit.java](https://plugins.gradle.org/plugin/org.shipkit.java)" Gradle plugin will automatically apply "[org.shipkit.bintray](https://plugins.gradle.org/plugin/org.shipkit.bintray) plugin what is not expected in this case.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This dobule quote: "[org.shipkit.bintray] is not closed anywhere.

Additionally, looking at this diff and nearby lines it seems more consistent to not enclose any clickable links in double quotes.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed

@mstachniuk mstachniuk merged commit b4fadaf into master Aug 23, 2019
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