Skip to content

Conversation

@milos-colic
Copy link
Contributor

Set provided dependencies.

Set provided dependencies.
@milos-colic milos-colic requested a review from nkvuong June 23, 2022 20:00
@codecov
Copy link

codecov bot commented Jun 23, 2022

Codecov Report

Merging #16 (47c9904) into main (c6bb662) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main      #16   +/-   ##
=========================================
  Coverage     88.88%   88.88%           
  Complexity       51       51           
=========================================
  Files             5        5           
  Lines           207      207           
  Branches         21       21           
=========================================
  Hits            184      184           
  Misses           14       14           
  Partials          9        9           
Impacted Files Coverage Δ
...tabricks/labs/delta/sharing/java/DeltaSharing.java 88.63% <ø> (ø)
...sharing/java/adaptor/DeltaSharingJsonProvider.java 88.23% <ø> (ø)
...aring/java/adaptor/DeltaSharingProfileAdaptor.java 100.00% <ø> (ø)
...ta/sharing/java/format/parquet/LocalInputFile.java 83.05% <ø> (ø)
...delta/sharing/java/format/parquet/TableReader.java 96.55% <ø> (ø)

Copy link
Contributor

@nkvuong nkvuong left a comment

Choose a reason for hiding this comment

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

Do we need to add author to pom.xml as well?

@nkvuong nkvuong merged commit 3df60d6 into main Jun 23, 2022
@nkvuong nkvuong deleted the feature/pom-fix branch June 23, 2022 20:43
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.

3 participants