ci: quote and group GITHUB_ENV writes#401
Conversation
📝 WalkthroughWalkthroughMultiple GitHub Actions workflows were edited to consolidate separate environment-variable echo/appends into single grouped blocks and to quote the environment file path (e.g., Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
Deploying maple with
|
| Latest commit: |
8cba5a2
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://5cf3b601.maple-ca8.pages.dev |
| Branch Preview URL: | https://chore-github-env-redirects.maple-ca8.pages.dev |
Greptile OverviewGreptile SummaryFixed ShellCheck warnings SC2086 and SC2129 across all GitHub Actions workflows by quoting Changes made:
Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant GHA as GitHub Actions
participant Shell as Shell Script
participant ENV as GITHUB_ENV File
Note over GHA,ENV: Before PR (Multiple Writes)
GHA->>Shell: Execute Configure sccache step
Shell->>ENV: echo "RUSTC_WRAPPER=sccache" >> $GITHUB_ENV
Shell->>ENV: echo "SCCACHE_DIR=$HOME/.cache/sccache" >> $GITHUB_ENV
Shell->>ENV: echo "SCCACHE_CACHE_SIZE=2G" >> $GITHUB_ENV
Note over Shell,ENV: ⚠️ Unquoted variable + 3 separate file operations
Note over GHA,ENV: After PR (Grouped & Quoted)
GHA->>Shell: Execute Configure sccache step
Shell->>ENV: { echo statements } >> "$GITHUB_ENV"
Note over Shell,ENV: ✅ Quoted variable + single grouped operation
ENV->>GHA: Environment variables available for next steps
|
Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
160b83a to
8cba5a2
Compare
Quotes "$GITHUB_ENV" and groups redirects for env writes across workflows (avoids ShellCheck SC2086/SC2129).
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.