Skip to content

Conversation

@gsmith-sas
Copy link
Member

No description provided.

@github-actions
Copy link
Contributor

sh-checker report

To get the full details, please check in the job output.

shellcheck errors

'shellcheck -e SC1004' returned error 1 finding the following syntactical issues:

----------

In logging/bin/deploy_opensearch_content.sh line 26:
tmpfile=$TMP_DIR/output.txt
^-----^ SC2034 (warning): tmpfile appears unused. Verify use (or export if used externally).

For more information:
  https://www.shellcheck.net/wiki/SC2034 -- tmpfile appears unused. Verify us...
----------

You can address the above issues in one of three ways:
1. Manually correct the issue in the offending shell script;
2. Disable specific issues by adding the comment:
  # shellcheck disable=NNNN
above the line that contains the issue, where NNNN is the error code;
3. Add '-e NNNN' to the SHELLCHECK_OPTS setting in your .yml action file.



shfmt errors
'shfmt -s -i 4 -bn -sr -ln bash' found no issues.

@gsmith-sas gsmith-sas changed the base branch from main to logging_lint4 August 28, 2025 14:40
@gsmith-sas gsmith-sas requested a review from vishaanS August 28, 2025 14:41
Copy link
Collaborator

@vishaanS vishaanS left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from logging_lint4 to main September 4, 2025 17:48
@gsmith-sas gsmith-sas marked this pull request as ready for review September 5, 2025 22:08
@gsmith-sas gsmith-sas merged commit c63002b into main Sep 9, 2025
2 checks passed
@gsmith-sas gsmith-sas deleted the content_proxy_fix branch September 9, 2025 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants