Skip to content
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

Don't publish jar signature artifacts for spock-bom #953

Merged
merged 2 commits into from
Nov 29, 2018

Conversation

marcphilipp
Copy link
Member

@marcphilipp marcphilipp commented Nov 21, 2018

Prior to this commit, spock-bom was using components.java
unnecessarily. As a side-effect, spock-bom-*.jar.asc was published
alongside the POM.


This PR was originally submitted as #951. However, the PR used the wrong base/target branch (stable-publishing instead of master).


This change is Reviewable

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Prior to this commit, spock-bom was using `components.java`
unnecessarily. As a side-effect, `spock-bom-*.jar.asc` was published
alongside the POM.
@codecov
Copy link

codecov bot commented Nov 21, 2018

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #953   +/-   ##
=========================================
  Coverage     75.99%   75.99%           
  Complexity     3534     3534           
=========================================
  Files           377      377           
  Lines         10742    10742           
  Branches       1367     1367           
=========================================
  Hits           8163     8163           
  Misses         2102     2102           
  Partials        477      477

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 070b7eb...d554b58. Read the comment docs.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Copy link
Member

@leonard84 leonard84 left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 2 of 2 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@leonard84 leonard84 merged commit 0fa757c into spockframework:master Nov 29, 2018
mkutz pushed a commit to mkutz/spock that referenced this pull request May 29, 2019
Prior to this commit, spock-bom was using `components.java`
unnecessarily. As a side-effect, `spock-bom-*.jar.asc` was published
alongside the POM.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants