Skip to content

ci(generate-theme-readme): align theme readme generation with TypeScript sources - #433

Merged
martin-mfg merged 1 commit into
stats-organization:masterfrom
marcalexiei:generate-theme
Aug 3, 2026
Merged

ci(generate-theme-readme): align theme readme generation with TypeScript sources#433
martin-mfg merged 1 commit into
stats-organization:masterfrom
marcalexiei:generate-theme

Conversation

@marcalexiei

Copy link
Copy Markdown

Follow-up to the themes/index.js ➡️ index.ts migration, which left the theme README generation pipeline pointing to files that no longer exist.

  • Renamed the workflow, script, and pnpm command to the consistent generate-theme-readme (previously generate-theme-doc / theme-readme-gen) and updated them to use the .ts source.
  • Reduced workflow permissions from 12 declared scopes to the 2 actually required:
    • contents: write
    • pull-requests: write
  • Fixed the generated PR body, which incorrectly referenced language files.

Note

After this is merged, I'd consider applying the same permission reduction and workflow rename to update-langs.yml.

@vercel

vercel Bot commented Aug 2, 2026

Copy link
Copy Markdown

@marcalexiei is attempting to deploy a commit to the martin-mfg's projects Team on Vercel.

A member of the Team first needs to authorize it.

@martin-mfg
martin-mfg merged commit df66b24 into stats-organization:master Aug 3, 2026
5 of 7 checks passed
@marcalexiei
marcalexiei deleted the generate-theme branch August 4, 2026 04:00
martin-mfg added a commit that referenced this pull request Aug 5, 2026
Like #433 for the update-langs workflow.
* rename the workflow, script, and pnpm command for consistency
* fix workflow which
[broke](https://github.com/stats-organization/github-stats-extended/actions/workflows/update-langs.yml)
after upgrading js-yaml a month ago
* reduced workflow permissions
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