wip#68
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #68 +/- ##
============================================
- Coverage 25.12% 25.10% -0.03%
Complexity 79 79
============================================
Files 54 54
Lines 979 980 +1
Branches 60 60
============================================
Hits 246 246
- Misses 725 726 +1
Partials 8 8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR addresses DD-2237 by extending the Dataverse client library with a file-metadata retrieval endpoint and updating file-ID-related APIs/models to use long-based identifiers (with a major version bump to reflect the breaking API change).
Changes:
- Bump project version to
2.0.0-SNAPSHOTto reflect breaking API changes. - Add
FileApi#getMetadata()for retrieving a file’s JSON representation/metadata. - Change file ID types from
inttolong/Longacross relevant models and client entry points (includingdeleteFiles).
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| pom.xml | Bumps parent build version to 2.0.0-SNAPSHOT. |
| lib/pom.xml | Bumps library module version to 2.0.0-SNAPSHOT. |
| examples/pom.xml | Bumps examples module version to 2.0.0-SNAPSHOT. |
| lib/src/main/java/nl/knaw/dans/lib/dataverse/model/file/DataFile.java | Updates file-related ID fields to long/Long. |
| lib/src/main/java/nl/knaw/dans/lib/dataverse/model/DataMessageWithId.java | Adjusts Lombok equality/hashCode generation to include superclass fields. |
| lib/src/main/java/nl/knaw/dans/lib/dataverse/FileApi.java | Adds getMetadata() API method. |
| lib/src/main/java/nl/knaw/dans/lib/dataverse/DataverseClient.java | Updates client entry points to accept long for file-related IDs. |
| lib/src/main/java/nl/knaw/dans/lib/dataverse/DatasetApi.java | Updates deleteFiles to accept List<Long>. |
| examples/src/main/java/nl/knaw/dans/lib/dataverse/example/FileGetMetadata.java | Adds runnable example for the new getMetadata() API. |
| examples/src/main/java/nl/knaw/dans/lib/dataverse/example/DatasetDeleteFiles.java | Updates example to use List<Long> file IDs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Fixes DD-2237
Description of changes
Notify
@DANS-KNAW/core-systems