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

style: config prettier and fix eslint error #2117

Merged
merged 1 commit into from
May 31, 2019
Merged

style: config prettier and fix eslint error #2117

merged 1 commit into from
May 31, 2019

Conversation

DiamondYuan
Copy link
Contributor

@DiamondYuan DiamondYuan commented May 30, 2019

Checklist
  • npm test passes
  • tests are included
  • documentation is changed or added
  • commit message follows commit guidelines
Description of change

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 87.186% when pulling 3874830 on DiamondYuan:master into c752cd9 on dvajs:master.

@coveralls
Copy link

coveralls commented May 30, 2019

Coverage Status

Coverage decreased (-0.1%) to 87.186% when pulling 5dd7a39 on DiamondYuan:master into c752cd9 on dvajs:master.

@sorrycc sorrycc merged commit 3f1f90e into dvajs:master May 31, 2019
@sorrycc
Copy link
Member

sorrycc commented May 31, 2019

👍

GoToBoy pushed a commit to GoToBoy/dva that referenced this pull request Oct 12, 2023
GoToBoy pushed a commit to GoToBoy/dva that referenced this pull request Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants