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

Update to the latest config #175

Merged
merged 10 commits into from
Apr 11, 2022
Merged

Update to the latest config #175

merged 10 commits into from
Apr 11, 2022

Conversation

yevhenii-nadtochii
Copy link
Contributor

@yevhenii-nadtochii yevhenii-nadtochii commented Apr 8, 2022

This PR updates the repository to use the latest config.

Mostly, this is done in order to pull in the following changes:

  1. Make published artifacts full-fledged #335.
  2. Bump dependencies #344.

Applying of those changes led to the following:

  1. testJar artifact has disappeared from the list of artifacts published by the modules. This artifact is optional now and not published by default.
  2. protoJar artifact has also disappeared in modules that don't have any proto definitions.

Please note, the repository does not use the latest version of ErrorProne and some Spine libraries. In particular mc-java. Take a look on #176 for details.

The library version is bumped to 2.0.0-SNAPSHOT.67.

@yevhenii-nadtochii yevhenii-nadtochii self-assigned this Apr 8, 2022
@codecov
Copy link

codecov bot commented Apr 8, 2022

Codecov Report

Merging #175 (ccf6355) into master (df06478) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #175   +/-   ##
=========================================
  Coverage     87.44%   87.44%           
  Complexity      462      462           
=========================================
  Files            74       74           
  Lines          1729     1729           
  Branches         81       81           
=========================================
  Hits           1512     1512           
  Misses          188      188           
  Partials         29       29           

@yevhenii-nadtochii yevhenii-nadtochii marked this pull request as ready for review April 11, 2022 11:11
@yevhenii-nadtochii
Copy link
Contributor Author

@armiol Please take a look.

Copy link
Contributor

@armiol armiol left a comment

Choose a reason for hiding this comment

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

LGTM with a few comments.

build.gradle.kts Outdated Show resolved Hide resolved
version.gradle.kts Outdated Show resolved Hide resolved
@yevhenii-nadtochii yevhenii-nadtochii merged commit 004294b into master Apr 11, 2022
@yevhenii-nadtochii yevhenii-nadtochii deleted the update-config branch April 11, 2022 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants