Skip to content

chore: add set -o pipefail to postCreateScript.sh - #113

Merged
gvatsal60 merged 5 commits into
masterfrom
chore/add-pipefail
Aug 5, 2026
Merged

gvatsal60 merged 5 commits into
masterfrom
chore/add-pipefail

Conversation

@gvatsal60

Copy link
Copy Markdown
Owner

Adds set -o pipefail to .devcontainer/postCreateScript.sh

Copilot AI lite review requested due to automatic review settings August 4, 2026 14:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

sh-checker report

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

shellcheck errors

'shellcheck ' returned error 1 finding the following syntactical issues:

----------

In .devcontainer/postCreateScript.sh line 2:
set -o pipefail
       ^------^ SC3040 (warning): In POSIX sh, set option pipefail is undefined.

For more information:
  https://www.shellcheck.net/wiki/SC3040 -- In POSIX sh, set option pipefail ...
----------

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 -w -i 4' returned error 1 finding the following formatting issues:

----------
diff .devcontainer/postCreateScript.sh.orig .devcontainer/postCreateScript.sh
--- .devcontainer/postCreateScript.sh.orig
+++ .devcontainer/postCreateScript.sh
@@ -45,7 +45,6 @@
     curl -fsSL --proto '=https' "$@"
 }
 
-
 ###################################################################################################
 # Main Script
 ###################################################################################################
@@ -71,4 +70,4 @@
             ln -s "$(realpath "${file}")" "${VSCODE_DIR}/$(basename "${file}")"
         done
     fi
-fi
\ No newline at end of file
+fi
----------

You can reformat the above files to meet shfmt's requirements by typing:

  shfmt -w -i 4 -w filename


Copilot AI review requested due to automatic review settings August 4, 2026 14:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

sh-checker report

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

shellcheck errors

'shellcheck ' returned error 1 finding the following syntactical issues:

----------

In .devcontainer/postCreateScript.sh line 2:
set -o pipefail
       ^------^ SC3040 (warning): In POSIX sh, set option pipefail is undefined.

For more information:
  https://www.shellcheck.net/wiki/SC3040 -- In POSIX sh, set option pipefail ...
----------

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 -w -i 4' returned error 1 finding the following formatting issues:

----------
diff .devcontainer/postCreateScript.sh.orig .devcontainer/postCreateScript.sh
--- .devcontainer/postCreateScript.sh.orig
+++ .devcontainer/postCreateScript.sh
@@ -45,7 +45,6 @@
     curl -fsSL --proto '=https' "$@"
 }
 
-
 ###################################################################################################
 # Main Script
 ###################################################################################################
----------

You can reformat the above files to meet shfmt's requirements by typing:

  shfmt -w -i 4 -w filename


@codacy-production

codacy-production Bot commented Aug 4, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Copilot AI review requested due to automatic review settings August 4, 2026 14:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

sh-checker report

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

shellcheck errors
'shellcheck ' found no issues.

shfmt errors

'shfmt -w -i 4' returned error 1 finding the following formatting issues:

----------
diff .devcontainer/postCreateScript.sh.orig .devcontainer/postCreateScript.sh
--- .devcontainer/postCreateScript.sh.orig
+++ .devcontainer/postCreateScript.sh
@@ -44,7 +44,6 @@
     curl -fsSL --proto '=https' "$@"
 }
 
-
 ###################################################################################################
 # Main Script
 ###################################################################################################
----------

You can reformat the above files to meet shfmt's requirements by typing:

  shfmt -w -i 4 -w filename


1 similar comment
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

sh-checker report

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

shellcheck errors
'shellcheck ' found no issues.

shfmt errors

'shfmt -w -i 4' returned error 1 finding the following formatting issues:

----------
diff .devcontainer/postCreateScript.sh.orig .devcontainer/postCreateScript.sh
--- .devcontainer/postCreateScript.sh.orig
+++ .devcontainer/postCreateScript.sh
@@ -44,7 +44,6 @@
     curl -fsSL --proto '=https' "$@"
 }
 
-
 ###################################################################################################
 # Main Script
 ###################################################################################################
----------

You can reformat the above files to meet shfmt's requirements by typing:

  shfmt -w -i 4 -w filename


Copilot AI review requested due to automatic review settings August 5, 2026 05:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

sh-checker report

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

shellcheck errors
'shellcheck ' found no issues.

shfmt errors

'shfmt -w -i 4' returned error 1 finding the following formatting issues:

----------
diff .devcontainer/postCreateScript.sh.orig .devcontainer/postCreateScript.sh
--- .devcontainer/postCreateScript.sh.orig
+++ .devcontainer/postCreateScript.sh
@@ -44,7 +44,6 @@
     curl -fsSL --proto '=https' "$@"
 }
 
-
 ###################################################################################################
 # Main Script
 ###################################################################################################
----------

You can reformat the above files to meet shfmt's requirements by typing:

  shfmt -w -i 4 -w filename


Copilot AI review requested due to automatic review settings August 5, 2026 14:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sonarqubecloud

sonarqubecloud Bot commented Aug 5, 2026

Copy link
Copy Markdown

@gvatsal60
gvatsal60 merged commit 4205f88 into master Aug 5, 2026
9 checks passed
@gvatsal60
gvatsal60 deleted the chore/add-pipefail branch August 5, 2026 14:26
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.

2 participants