-
Notifications
You must be signed in to change notification settings - Fork 1
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
Dcat 2.1.0 #85
Dcat 2.1.0 #85
Conversation
Reviewer's Guide by SourceryThis PR updates the DCAT extension to version 2.1.0 and implements changes to simplify the codebase by removing redundant test cases and updating the metadata field handling. The implementation focuses on streamlining the contact point and creator handling while updating dependencies across the project. ER diagram for updated dataset fieldserDiagram
DATASET {
string has_version
string issued
string modified
string temporal_start
string temporal_end
}
note for DATASET "Removed fields: creator"
Updated class diagram for FAIRDataPointDCATAPProfileclassDiagram
class FAIRDataPointDCATAPProfile {
+parse_dataset(dataset_dict: Dict, dataset_ref: URIRef) Dict
+validate_tags(tags: List) List
}
note for FAIRDataPointDCATAPProfile "Removed methods: _contact_point_details, _parse_contact_point, _parse_creator"
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @hcvdwerf - I've reviewed your changes - here's some feedback:
Overall Comments:
- Consider adding test coverage for the enhanced publisher and contact metadata fields to ensure the new functionality works as expected.
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
🚀 Pull Request Checklist
Title:
[x]
Upgrade to DCAT 2.1.0Description:
[x]
Upgrade to DCAT 2.1.0 where contact, creator and publisher are correctly mappedContext:
[x]
Changed are made to keep changes as much as possible upstreamChanges:
[x ]
Removed obsolete test cases. Upgrade to 2.1.0Testing:
[x ]
old test cases which are now removed still works.Checklist:
[x ]
I have checked that my code adheres to the project's style guidelines and that my code is well-commented.[ x]
I have performed self-review of my own code and corrected any misspellings.[ x]
I have made corresponding changes to the documentation (if applicable).[ x]
My changes generate no new warnings or errors.[ x]
I have added tests that prove my fix is effective or that my feature works.[ x]
New and existing unit tests pass locally with my changes.Summary by Sourcery
Enhance dataset parsing by updating metadata fields, remove redundant tests, update documentation for new dependencies, and adjust CI workflow to use updated dependencies.
Enhancements:
CI:
Documentation:
Tests: