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

feat: rename logrotater => logrotator #4

Merged
merged 2 commits into from
Aug 17, 2016
Merged

feat: rename logrotater => logrotator #4

merged 2 commits into from
Aug 17, 2016

Conversation

popomore
Copy link
Member

@popomore popomore commented Aug 16, 2016

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

整个库

Description of change

改名后需要重新 publish 一个新库

eggjs/egg#49

@mention-bot
Copy link

@popomore, thanks for your PR! By analyzing the annotation information on this pull request, we identified @fengmk2 and @jtyjty99999 to be potential reviewers

@popomore
Copy link
Member Author

@fengmk2 同步下 appveyor

@codecov-io
Copy link

codecov-io commented Aug 16, 2016

Current coverage is 98.19% (diff: 100%)

No coverage report found for master at 7793b2f.

Powered by Codecov. Last update 7793b2f...d2abad6

@popomore
Copy link
Member Author

codecov 的算法太高级

@popomore
Copy link
Member Author

appveyor 我建了个新的

@popomore
Copy link
Member Author

popomore commented Aug 16, 2016

补全测试覆盖率了

@jtyjty99999
Copy link
Member

+1 ..

@popomore
Copy link
Member Author

那这个我合了

@popomore popomore merged commit 0fe84da into master Aug 17, 2016
@popomore popomore deleted the rename branch August 17, 2016 04:20
@popomore
Copy link
Member Author

  • egg-logrotator@2.0.0

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.

5 participants