-
Notifications
You must be signed in to change notification settings - Fork 5
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
Feature/ted 713 #293
Feature/ted 713 #293
Conversation
✅ Linked to Story TED-713 · Mapping suite diff |
Codecov ReportBase: 98.85% // Head: 98.99% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #293 +/- ##
==========================================
+ Coverage 98.85% 98.99% +0.14%
==========================================
Files 103 106 +3
Lines 5233 5486 +253
==========================================
+ Hits 5173 5431 +258
+ Misses 60 55 -5
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
ted_sws/core/adapters/cmd_runner.py
Outdated
@@ -63,6 +63,29 @@ def __init__(self, name=__name__, logger: EventLogger = None): | |||
|
|||
self.logger = logger | |||
|
|||
@classmethod | |||
def _init_list_input_opts_split(cls, input_val) -> List: |
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.
use typing for input_val
@@ -46,6 +46,8 @@ def __init__( | |||
repository_path = Path(self.mappings_path) | |||
mapping_suite_repository = MappingSuiteRepositoryInFileSystem(repository_path=repository_path) | |||
self.mapping_suite = mapping_suite_repository.get(reference=self.mapping_suite_id) | |||
self.log("K :: " + str(repository_path)) |
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.
what is this? 👯
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.
:D some development radiation. Thanks for observing
SonarCloud Quality Gate failed. |
No description provided.