Skip to content

ci: avoid expanding secrets in run blocks#24633

Merged
Artur- merged 1 commit into
mainfrom
fix-secrets-expand
Jun 16, 2026
Merged

ci: avoid expanding secrets in run blocks#24633
Artur- merged 1 commit into
mainfrom
fix-secrets-expand

Conversation

@Artur-

@Artur- Artur- commented Jun 15, 2026

Copy link
Copy Markdown
Member

Pass TB_LICENSE through step-level env vars and reference it as a shell variable instead of expanding ${{ secrets.TB_LICENSE }} directly into the run scripts, addressing the secret-expansion workflow warning.

Pass TB_LICENSE through step-level env vars and reference it as a shell
variable instead of expanding ${{ secrets.TB_LICENSE }} directly into the
run scripts, addressing the secret-expansion workflow warning.
@Artur- Artur- requested a review from manolo June 15, 2026 15:41
@sonarqubecloud

sonarqubecloud Bot commented Jun 15, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@github-actions

Copy link
Copy Markdown

Test Results

 1 451 files  ±0   1 451 suites  ±0   1h 25m 31s ⏱️ -33s
10 249 tests ±0  10 181 ✅ ±0  68 💤 ±0  0 ❌ ±0 
10 721 runs  ±0  10 652 ✅ ±0  69 💤 ±0  0 ❌ ±0 

Results for commit b8c3031. ± Comparison against base commit 4d08333.

@Artur- Artur- added this pull request to the merge queue Jun 15, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 15, 2026
@Artur- Artur- added this pull request to the merge queue Jun 15, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 15, 2026
@Artur- Artur- added this pull request to the merge queue Jun 16, 2026
Merged via the queue into main with commit bd1879a Jun 16, 2026
34 checks passed
@Artur- Artur- deleted the fix-secrets-expand branch June 16, 2026 05:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants