8000 Fix run dev by Triumph-light · Pull Request #2778 · alibaba/hooks · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix run dev #2778

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix run dev #2778

wants to merge 2 commits into from

Conversation

Triumph-light
Copy link

[中文版模板 / Chinese template]

🤔 This is a ...

  • New feature
  • [✅] Bug fix
  • Site / documentation update
  • Demo update
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Internationalization
  • Refactoring
  • Code style optimization
  • Test Case
  • Branch merge
  • Other (about what?)

🔗 Related issue link

Fixes #xxx

💡 Background and solution

在 macOS 系统上执行 dev 会因 set NODE_OPTIONS=--openssl-legacy-provider 设置方式不兼容而启动失败,并且无法让配置生效。
为解决跨平台环境变量设置问题,采用 cross-env 工具,确保 NODE_OPTIONS 能在 Windows、macOS 和 Linux 上统一生效。

改动内容:

  • 修改 package.json 中的 dev 命令为:
    "dev": "cross-env NODE_OPTIONS=--openssl-legacy-provider dumi dev"

📝 Changelog

Language Changelog
🇺🇸 English Fix dev command failing on Windows by adding cross-env.
🇨🇳 Chinese 修复 Windows 下 dumi dev 启动失败的问题,增加 cross-env 依赖。

☑️ Self Check before Merge

  • [✅] 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

@CLAassistant
Copy link
CLAassistant commented May 8, 2025

CLA assistant check
All committers have signed the CLA.

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
0