Skip to content

support variables in script; change argument release to version#143

Merged
franknli merged 11 commits into
mainfrom
fli/support_script_variables
Oct 21, 2024
Merged

support variables in script; change argument release to version#143
franknli merged 11 commits into
mainfrom
fli/support_script_variables

Conversation

@franknli

Copy link
Copy Markdown
Collaborator

No description provided.

@franknli franknli requested review from a team and PipKat as code owners October 18, 2024 14:09
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Oct 18, 2024
@wiz-inc-572fc38784

wiz-inc-572fc38784 Bot commented Oct 18, 2024

Copy link
Copy Markdown

Wiz Scan Summary

Scan Module Critical High Medium Low Info Total
IaC Misconfigurations 0 0 0 0 0 0
Vulnerabilities 0 0 0 0 0 0
Sensitive Data 0 0 0 0 1 1
Secrets 0 0 0 0 0 0
Total 0 0 0 0 1 1

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

@github-actions github-actions Bot added the testing Anything related to tests label Oct 18, 2024
@franknli franknli requested a review from asawas October 18, 2024 19:26
Variables in the script specified as $$varname%%50%% will be converted to variable
values or use the default value - 50 in the example.
log_level : str, default: "error"
Level of logging. Options are "critical" "debug", "error", "info", and "warning".

@asawas asawas Oct 21, 2024

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Options are "critical", "debug", "error", "info", and "warning".
add a comma between "critical" and "debug". Also the same on line 150

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

self,
show_gui=True,
release="242",
version="242",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to update the default version to "251" ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not yet.

@franknli franknli merged commit ba91f28 into main Oct 21, 2024
@franknli franknli deleted the fli/support_script_variables branch October 21, 2024 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation testing Anything related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants