Skip to content

ci: pass GITHUB_TOKEN to license attribution step#7140

Merged
watson merged 1 commit intomasterfrom
watson/fix-3rdparty-license-check
Dec 18, 2025
Merged

ci: pass GITHUB_TOKEN to license attribution step#7140
watson merged 1 commit intomasterfrom
watson/fix-3rdparty-license-check

Conversation

@watson
Copy link
Copy Markdown
Collaborator

@watson watson commented Dec 18, 2025

What Does This Do?

Fixes the update-3rdparty-licenses workflow by explicitly passing the GITHUB_TOKEN to the license attribution step.

Motivation

The workflow was failing with this error:

Invalid value for '--github-token' (env var: 'GITHUB_TOKEN'): No Github token available.

While the GITHUB_TOKEN is automatically available in GitHub Actions, it needs to be explicitly passed as an environment variable to individual steps that require it.

Copy link
Copy Markdown
Collaborator Author

watson commented Dec 18, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 18, 2025

Overall package size

Self size: 4.35 MB
Deduped: 5.24 MB
No deduping: 5.24 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 1.15.0 | 127.66 kB | 856.24 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |

🤖 This report was automatically generated by heaviest-objects-in-the-universe

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.78%. Comparing base (8f86699) to head (8044a58).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7140      +/-   ##
==========================================
- Coverage   84.79%   84.78%   -0.01%     
==========================================
  Files         521      521              
  Lines       22159    22159              
==========================================
- Hits        18790    18788       -2     
- Misses       3369     3371       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pr-commenter
Copy link
Copy Markdown

pr-commenter Bot commented Dec 18, 2025

Benchmarks

Benchmark execution time: 2025-12-18 11:31:54

Comparing candidate commit 8044a58 in PR branch watson/fix-3rdparty-license-check with baseline commit 8f86699 in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 289 metrics, 31 unstable metrics.

The dd-license-attribution tool requires a GitHub token to avoid
API rate limits. This change explicitly passes the GITHUB_TOKEN
as an environment variable to the license regeneration step.
@watson watson force-pushed the watson/fix-3rdparty-license-check branch from ab548a7 to 8044a58 Compare December 18, 2025 11:14
@watson watson self-assigned this Dec 18, 2025
@watson watson marked this pull request as ready for review December 18, 2025 11:16
@watson watson requested a review from a team as a code owner December 18, 2025 11:16
@watson watson enabled auto-merge (squash) December 18, 2025 11:39
@watson watson merged commit 7c979d2 into master Dec 18, 2025
1535 of 1553 checks passed
@watson watson deleted the watson/fix-3rdparty-license-check branch December 18, 2025 13:24
dd-octo-sts Bot pushed a commit that referenced this pull request Dec 19, 2025
The dd-license-attribution tool requires a GitHub token to avoid
API rate limits. This change explicitly passes the GITHUB_TOKEN
as an environment variable to the license regeneration step.
@dd-octo-sts dd-octo-sts Bot mentioned this pull request Dec 19, 2025
nina9753 pushed a commit that referenced this pull request Dec 20, 2025
The dd-license-attribution tool requires a GitHub token to avoid
API rate limits. This change explicitly passes the GITHUB_TOKEN
as an environment variable to the license regeneration step.
nina9753 pushed a commit that referenced this pull request Jan 15, 2026
The dd-license-attribution tool requires a GitHub token to avoid
API rate limits. This change explicitly passes the GITHUB_TOKEN
as an environment variable to the license regeneration step.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants