Skip to content

ci: adopt uppt for release workflow - #866

Merged
danielroe merged 1 commit into
mainfrom
ci/uppt
Aug 21, 2026
Merged

danielroe merged 1 commit into
mainfrom
ci/uppt

Conversation

@danielroe

Copy link
Copy Markdown
Member

πŸ”— Linked issue

πŸ“š Description

@pkg-pr-new

pkg-pr-new Bot commented Aug 21, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@nuxt/fonts@866

commit: 15cb763

@danielroe
danielroe enabled auto-merge August 21, 2026 19:55
@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. πŸŽ‰

ℹ️ Recent review info
βš™οΈ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d27dbd38-8af0-4093-b4b9-7c70076b5806

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between b38d6f2 and 15cb763.

β›” Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
πŸ“’ Files selected for processing (7)
  • .github/workflows/changelogensets.yml
  • .github/workflows/pkg-pr-new.yml
  • .github/workflows/release-nightly.yml
  • .github/workflows/release.yml
  • package.json
  • scripts/_utils.ts
  • scripts/update-changelog.ts
πŸ’€ Files with no reviewable changes (5)
  • package.json
  • .github/workflows/release-nightly.yml
  • scripts/update-changelog.ts
  • scripts/_utils.ts
  • .github/workflows/changelogensets.yml

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.


πŸ“ Walkthrough

Walkthrough

The change adds a preview publishing workflow for pushes and pull requests targeting main. The release workflow now separates pull request release handling, merged release processing, package creation, and tagged publishing. The change removes the former changelog and nightly release workflows, deletes their supporting scripts, and removes semantic-versioning and changelog development dependencies.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Merge Risk: βšͺ Minimal Β· up to 15cb7

No actionable merge-blocking risk remains based on the supplied evidence; the PR is merge-ready after normal checks and review.

πŸš₯ Pre-merge checks | βœ… 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description contains only the default template and does not explain the release workflow migration. Add a brief description of the Uppt migration, workflow changes, removed automation, and any linked issue or testing details.
βœ… Passed checks (4 passed)
Check name Status Explanation
Title check βœ… Passed The title clearly identifies the main change: adopting Uppt for the release workflow.
Docstring Coverage βœ… Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)
Linked Issues check βœ… Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check βœ… Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
πŸ“ Generate docstrings
  • Create stacked PR
  • Commit on current branch
πŸ§ͺ Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/uppt

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❀️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

βœ… All modified and coverable lines are covered by tests.
βœ… Project coverage is 68.01%. Comparing base (b38d6f2) to head (15cb763).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #866   +/-   ##
=======================================
  Coverage   68.01%   68.01%           
=======================================
  Files           9        9           
  Lines         297      297           
  Branches       67       67           
=======================================
  Hits          202      202           
  Misses         78       78           
  Partials       17       17           

β˜” View full report in Codecov by Harness.
πŸ“’ 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.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying fonts with Β Cloudflare Pages Β Cloudflare Pages

Latest commit: 15cb763
Status:Β βœ…Β  Deploy successful!
Preview URL: https://31fb2b34.fonts-dsw.pages.dev
Branch Preview URL: https://ci-uppt.fonts-dsw.pages.dev

View logs

@danielroe
danielroe added this pull request to the merge queue Aug 21, 2026
Merged via the queue into main with commit 4871025 Aug 21, 2026
13 checks passed
@danielroe
danielroe deleted the ci/uppt branch August 21, 2026 20:05
This was referenced Aug 21, 2026
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