Conversation
There was a problem hiding this comment.
Pull request overview
This pull request updates dependencies and modifies build script commands in the create-next2d-app project. The version is bumped from 2.1.14 to 2.1.16, indicating a minor release.
Changes:
- Updated multiple dependencies including @types/node (24→25), fs-extra, validate-npm-package-name, and several ESLint-related packages
- Removed
--env prdflag from all build and open commands in generated package.json scripts - Added Node.js version specification and npm update step to GitHub Actions workflow
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| package.json | Version bump to 2.1.16 and dependency updates |
| package-lock.json | Lock file updates matching package.json changes, fixes peer dependency markers |
| src/index.ts | Removed --env prd flag from build/open scripts and reordered commands |
| .github/workflows/lint.yml | Added Node.js version 24 specification and npm upgrade step |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "open:ios": "npx @next2d/builder --platform ios --open", | ||
| "open:android": "npx @next2d/builder --platform android --open", | ||
| "build:steam:windows": "npx @next2d/builder --platform steam:windows", | ||
| "build:steam:macos": "npx @next2d/builder --platform steam:macos", | ||
| "build:steam:linux": "npx @next2d/builder --platform steam:linux", | ||
| "build:web": "npx @next2d/builder --platform web", | ||
| "build:ios": "npx @next2d/builder --platform ios --build", | ||
| "build:android": "npx @next2d/builder --platform android --build", | ||
| "build": "npx @next2d/builder", |
There was a problem hiding this comment.
The removal of the --env prd flag from build and open commands is a significant behavioral change. This changes how the @next2d/builder tool will be invoked for production builds. If the builder tool defaults to a different environment when --env is not specified, this could result in builds not being configured for production. Please verify this is intentional and document the expected behavior change.
| "open:ios": "npx @next2d/builder --platform ios --open", | |
| "open:android": "npx @next2d/builder --platform android --open", | |
| "build:steam:windows": "npx @next2d/builder --platform steam:windows", | |
| "build:steam:macos": "npx @next2d/builder --platform steam:macos", | |
| "build:steam:linux": "npx @next2d/builder --platform steam:linux", | |
| "build:web": "npx @next2d/builder --platform web", | |
| "build:ios": "npx @next2d/builder --platform ios --build", | |
| "build:android": "npx @next2d/builder --platform android --build", | |
| "build": "npx @next2d/builder", | |
| "open:ios": "npx @next2d/builder --platform ios --open --env prd", | |
| "open:android": "npx @next2d/builder --platform android --open --env prd", | |
| "build:steam:windows": "npx @next2d/builder --platform steam:windows --env prd", | |
| "build:steam:macos": "npx @next2d/builder --platform steam:macos --env prd", | |
| "build:steam:linux": "npx @next2d/builder --platform steam:linux --env prd", | |
| "build:web": "npx @next2d/builder --platform web --env prd", | |
| "build:ios": "npx @next2d/builder --platform ios --build --env prd", | |
| "build:android": "npx @next2d/builder --platform android --build --env prd", | |
| "build": "npx @next2d/builder --env prd", |
No description provided.