Skip to content
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

fix: run "rush change-all" failed (via NodeJs CVE-2024-27980) (#3303) #3307

Closed
wants to merge 1 commit into from

Conversation

depux
Copy link

@depux depux commented Jan 10, 2025

[中文版模板 / Chinese template]

🤔 This is a ...

  • New feature
  • Bug fix
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Refactoring
  • Update dependency
  • Code style optimization
  • Test Case
  • Branch merge
  • Site / documentation update
  • Demo update
  • Workflow
  • Chore
  • Release
  • Other (about what?)

🔗 Related issue link

1、close #3303
2、causes of bug:https://nodejs.org/en/blog/vulnerability/april-2024-security-releases-2

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English
🇨🇳 Chinese

☑️ Self-Check before Merge

⚠️ Please check all items below before requesting a reviewing. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

🚀 Summary

copilot:summary

🔍 Walkthrough

copilot:walkthrough

@github-actions github-actions bot added the chore label Jan 10, 2025
@depux depux closed this Jan 10, 2025
@depux
Copy link
Author

depux commented Jan 10, 2025

  1. reference: [Chore] git commit or push might throw error in Windows environment VChart#3622
  2. common/scripts/install-run.js is automatically generated by rush

@depux depux reopened this Jan 10, 2025
@fangsmile
Copy link
Contributor

这个文件common/scripts/install-run.js 是rush update 的时候给生成的 所以更正是无效的

@fangsmile fangsmile closed this Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] run ”rush change-all“ failed:Unable to install package: Error: "npm install" encountered an error
2 participants