-
Notifications
You must be signed in to change notification settings - Fork 23
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
Enhance import_map functionality to support namespace as well as indi… #162
Conversation
…vidual file mapping. Also separated out import mapping code so that linkml can use the same code as the utilities.
Codecov Report
@@ Coverage Diff @@
## main #162 +/- ##
==========================================
+ Coverage 62.54% 63.81% +1.26%
==========================================
Files 49 49
Lines 5219 5217 -2
Branches 1468 1463 -5
==========================================
+ Hits 3264 3329 +65
+ Misses 1593 1523 -70
- Partials 362 365 +3
Continue to review full report at Codecov.
|
@cmungall - I need this to be in pypi before I can commit a corresponding linkml change. What is the process these days? |
same as before - just make a new github release, actions will propagate to pypi See the docs: https://github.com/linkml/linkml-runtime/blob/main/CONTRIBUTING.md#release-process |
@@ -50,6 +50,17 @@ def to_file_uri(fname: str) -> str: | |||
JsonObj(**{"@context": context_list[0] if len(context_list) == 1 else context_list}) | |||
|
|||
|
|||
def map_import(importmap: Dict[str, str], namespaces: Callable[[None], "Namespaces"], imp: Any) -> str: | |||
sname = str(imp) |
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.
a docstring would be great here
…vidual file mapping.
Also separated out import mapping code so that linkml can use the same code as the utilities.