-
Notifications
You must be signed in to change notification settings - Fork 4
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
Features/refactoring #6
Conversation
* Implement internal structure to hold OEP metadata * Implement compiler to translate internal structure to OEP Metadata v1.4
Python3.5 does not support it and we do not really use it
Some license ids end with `+` this leads to errors in license handling
A renderer takes a structure used in a dialect (e.g. JSON, RDF) and translates it into a string
fields did not have a resource in foreign keys
The translation to rdf loses the ordering information for all lists in the metadata. See #5 for more information
* introduce load method to load string into input structure (e.g. JSON, RDF-graph) * parse() expects those structures * parse_from_string covers the former string functionality
Closes #1 |
Closes #2 |
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.
Wow. That's a lot of new code. I'm glad I saw that some functions survived. Build and installation seem to work, tests pass in tox, but to be honest, I wasn't able to figure out how to actually use the tool now.
|
||
|
||
def main(): | ||
cli() |
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.
I'd just like to state and thereby express my appreciation of how clean this looks.
"BSL-1.0": "https://spdx.org/licenses/BSL-1.0.html", | ||
"Bahyph": "https://spdx.org/licenses/Bahyph.html", | ||
"Barr": "https://spdx.org/licenses/Barr.html", | ||
"Beerware": "https://spdx.org/licenses/Beerware.html", |
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.
I'm really glad to find this included.
This pull request introduces a python-based internal structure for OEP metadata and four major concepts