Skip to content

fix(ci): add set -euo pipefail to dependency update workflow scripts#6888

Closed
Copilot wants to merge 2 commits intocopilot/add-docs-mcp-to-productsfrom
copilot/sub-pr-6885-again
Closed

fix(ci): add set -euo pipefail to dependency update workflow scripts#6888
Copilot wants to merge 2 commits intocopilot/add-docs-mcp-to-productsfrom
copilot/sub-pr-6885-again

Conversation

Copy link
Contributor

Copilot AI commented Feb 27, 2026

Addresses review feedback on #6885 requesting stricter bash error handling in the dependency update workflow.

Changes

  • Added set -euo pipefail to both script blocks in .github/workflows/check-pinned-deps.yml
    • "Check for update" step (version extraction and comparison)
    • "Update pinned version" step (sed-based file modification)

This ensures scripts fail fast on:

  • Any command failure (-e)
  • Unset variable references (-u)
  • Pipeline failures (-o pipefail)

Works in conjunction with existing explicit empty-value checks to prevent silent failures when regex patterns stop matching or API calls fail.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…orkflow

Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix InfluxDB documentation and content links fix(ci): add set -euo pipefail to dependency update workflow scripts Feb 27, 2026
@jstirnaman
Copy link
Contributor

Duplicate of #6887, both superseded by commit 86702b4 on #6885.

@jstirnaman jstirnaman closed this Feb 28, 2026
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