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: Update utils.js fix the run error #33

Merged
merged 1 commit into from
Nov 1, 2017
Merged

Conversation

icarusion
Copy link
Contributor

没有用到,会直接报错

@jiandansousuo-bot
Copy link
Collaborator

hi @icarusion,非常感谢您的 PR ,但是您没有使用 PR 标题规则 格式,请及时修改, 谢谢!

@xuexb
Copy link
Owner

xuexb commented Oct 31, 2017

赞~

这是我的疏忽,我在运行时忘记删除 node_modules 目录,导致我升级 #25 后没有报错。请 @icarusion 根据 PR 标题规则 修改下标题,我将合入此次 PR ~ 感谢~

@xuexb xuexb changed the title Update utils.js fix: Update utils.js fix the run error Nov 1, 2017
@jiandansousuo-bot
Copy link
Collaborator

hi @icarusion,非常感谢您及时修正标题格式,祝您玩的开心!

@xuexb xuexb merged commit 37b87de into xuexb:master Nov 1, 2017
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.

3 participants