-
Notifications
You must be signed in to change notification settings - Fork 5
ci: quote and group GITHUB_ENV writes #401
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.