Skip to content

chore: add commit hash and pr link#13

Merged
yifancong merged 1 commit intomainfrom
fix/dependencies
Nov 21, 2025
Merged

chore: add commit hash and pr link#13
yifancong merged 1 commit intomainfrom
fix/dependencies

Conversation

@yifancong
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings November 21, 2025 07:59
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 21, 2025

Rsdoctor Bundle Diff Analysis

Found 2 project(s) in monorepo.

📁 rsbuild-demo

Path: examples/rsbuild-demo/dist/.rsdoctor/rsdoctor-data.json

📌 Baseline Commit: ae336e4a61 | PR: #12

Metric Current Baseline Change
📊 Total Size 190.1 KB 190.1 KB 0 B (0.0%)
📄 JavaScript 188.2 KB 188.2 KB 0 B (0.0%)
🎨 CSS 345.0 B 345.0 B 0 B (0.0%)
🌐 HTML 374.0 B 374.0 B 0 B (0.0%)
📁 Other Assets 1.2 KB 1.2 KB 0 B (0.0%)

📦 Download Diff Report: rsbuild-demo Bundle Diff

📁 rsbuild-demo2

Path: examples/rsbuild-demo2/dist/.rsdoctor/rsdoctor-data.json

📌 Baseline Commit: ae336e4a61 | PR: #12

Metric Current Baseline Change
📊 Total Size 190.7 KB 190.7 KB 0 B (0.0%)
📄 JavaScript 188.8 KB 188.8 KB 0 B (0.0%)
🎨 CSS 345.0 B 345.0 B 0 B (0.0%)
🌐 HTML 374.0 B 374.0 B 0 B (0.0%)
📁 Other Assets 1.2 KB 1.2 KB 0 B (0.0%)

📦 Download Diff Report: rsbuild-demo2 Bundle Diff

Generated by Rsdoctor GitHub Action

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request makes two main changes: (1) moves fast-glob from dependencies to devDependencies in package.json, and (2) adds functionality to track and display baseline commit information along with associated PRs in bundle analysis reports.

  • Dependency reorganization: fast-glob moved to devDependencies since it's only used during build/development
  • New feature: Display baseline commit hash and associated PR links in bundle analysis reports
  • Implementation: Added findPRsByCommit method to GitHubService to fetch PRs associated with baseline commits

Reviewed Changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
package.json Moved fast-glob from dependencies to devDependencies
pnpm-lock.yaml Updated lock file to reflect dependency reorganization
src/github.ts Added findPRsByCommit method to fetch PRs associated with a commit
src/report.ts Updated report generation to include baseline commit and PR information
src/index.ts Integrated baseline PR tracking into the main workflow
dist/index.js Compiled output reflecting all source code changes including bundled fast-glob dependencies
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yifancong yifancong changed the title Fix/dependencies chore: add commit hash and pr link Nov 21, 2025
@yifancong yifancong merged commit 44c54ca into main Nov 21, 2025
2 checks passed
@yifancong yifancong deleted the fix/dependencies branch November 21, 2025 08:04
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