-
Notifications
You must be signed in to change notification settings - Fork 55
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
fix: default set-parameter values as list #1438
Conversation
Signed-off-by: Lou DeGenaro <lou.degenaro@gmail.com>
@@ -576,14 +576,27 @@ def _get_control_implementation( | |||
component.control_implementations.append(control_implementation) | |||
return control_implementation | |||
|
|||
def _str_to_list(self, value: str) -> List[str]: | |||
"""List to string.""" | |||
rval = [] |
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.
The comment should be string to 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.
Fixed.
trestle/tasks/csv_to_oscal_cd.py
Outdated
# remove whitespace | ||
for v in values: | ||
n = v.replace(' ', '') | ||
n = n.strip() |
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 the purpose of both the lines above? Just to remove leading and trailing spaces, or also remove spaces in between? Suppose a single value is a string containing 2 or more words. This will remove spaces in between words also. Is that the intention?
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.
Fixed.
Signed-off-by: Lou DeGenaro <lou.degenaro@gmail.com>
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.
LGTM
* fix: default set-parameter values as list Signed-off-by: Lou DeGenaro <lou.degenaro@gmail.com> * fix string to list comment and blanks removal Signed-off-by: Lou DeGenaro <lou.degenaro@gmail.com> --------- Signed-off-by: Lou DeGenaro <lou.degenaro@gmail.com>
Types of changes
develop
->main
)Quality assurance (all should be covered).
Summary
Key links:
Before you merge