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: windows build 分隔符处理 / #31

Merged
merged 1 commit into from
Mar 5, 2021
Merged

Conversation

ynwshy
Copy link
Contributor

@ynwshy ynwshy commented Feb 8, 2021

Why

windows 环境下 执行 npm run build 会解析路径失败
linux 下能正常执行
需要对window分隔符处理 ‘\’ 替换 ‘/’

@auto-add-label auto-add-label bot added the bug Something isn't working label Feb 8, 2021
@2nthony
Copy link
Member

2nthony commented Feb 8, 2021

如果能加一张在 Windows 运行成功截图那就更好了

@levy9527
Copy link
Contributor

levy9527 commented Feb 8, 2021

兄弟,来张截图吧,好合并发版本

@ynwshy
Copy link
Contributor Author

ynwshy commented Feb 18, 2021

  • windows 下打包回出现路径解析问题
    image
  • 修改utils/index.js 增加如下代码
    image

@2nthony
Copy link
Member

2nthony commented Feb 18, 2021 via email

@ynwshy
Copy link
Contributor Author

ynwshy commented Feb 18, 2021

我直接改项目下的 node_modules/@femessage/update-popup/utils/index.js
执行npm run build 本地 打包没报错
image

@levy9527 levy9527 merged commit bdf1d6e into FEMessage:dev Mar 5, 2021
@levy9527
Copy link
Contributor

levy9527 commented Mar 5, 2021

@all-contributors add @ynwshy bug

@allcontributors
Copy link
Contributor

@levy9527

I've put up a pull request to add @ynwshy! 🎉

@levy9527
Copy link
Contributor

levy9527 commented Mar 5, 2021

@ynwshy 你好,已发布npm,并把你添加为了 贡献者。 这么迟才发,不好意思🤝

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants