-
Notifications
You must be signed in to change notification settings - Fork 2
[optimize] upgrade GitHub copilot instruction #10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ture Co-authored-by: TechQuery <19969570+TechQuery@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot 修改一下这些细节
…ormatting Co-authored-by: TechQuery <19969570+TechQuery@users.noreply.github.com>
Updated the copilot instructions with all requested details:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot 通篇修改相同问题
.github/copilot-instructions.md
Outdated
| - Install dependencies: `pnpm install` (requires Node.js, takes ~1-2 minutes) | ||
| - Start development server: `pnpm dev` (starts Parcel dev server in ~5-10 seconds, NEVER CANCEL) | ||
| - Access application: `http://localhost:1234` (Parcel default port) | ||
| - **Build for production**: `pnpm run build` (creates optimized bundle with PWA support) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| - **Build for production**: `pnpm run build` (creates optimized bundle with PWA support) | |
| - **Build for production**: `pnpm build` (creates optimized bundle with PWA support) |
项目中的 NPM scripts 作者通常不会与现有子命令重名,所以可以用简写。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated to use shorthand format pnpm build instead of pnpm run build throughout the file. Applied the same pattern to all PNPM commands for consistency. Commit: 4f99aa6
…uctions Co-authored-by: TechQuery <19969570+TechQuery@users.noreply.github.com>
根据代码库最新状态和 PR review comments,更新了
.github/copilot-instructions.md文件以准确反映当前的现代化技术栈,并优先使用 PNPM 包管理器。主要变更
包管理器优先级调整
pnpm install、pnpm dev、pnpm test等)文档格式改进
mobx-github)以提高可读性pnpm build而非pnpm run build)技术栈完整更新
mobx-github包用于 GitHub REST API v3(计划迁移到 OAuth token 系统)开发体验优化
所有命令和验证步骤都已在当前代码库中测试通过,确保指导文件与实际项目状态完全一致。
Fixes #9.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.