Skip to content
This repository has been archived by the owner on Jun 30, 2019. It is now read-only.

Refactor protocol #20

Closed
wants to merge 28 commits into from
Closed

Refactor protocol #20

wants to merge 28 commits into from

Conversation

cosven
Copy link
Owner

@cosven cosven commented Aug 6, 2018

No description provided.

@coveralls
Copy link

coveralls commented Aug 6, 2018

Coverage Status

Coverage increased (+3.6%) to 57.387% when pulling 8805d76 on refactor-protocol into 162c905 on master.

cosven and others added 24 commits August 6, 2018 11:10
Enum is bad for integration with other programs.
See: https://docs.python.org/3/library/enum.html#comparisons

在调试的时候,我们想改变 Player 的状态,直接的想法是: player.state = 3
而之前使用 Enum 的话,就不能这么直接的完成,需要 player.state = State.xxx

在真正的代码中,更推荐使用后者,但是在调试时,前者方便许多,所以将 Enum 改成 IntEnum
* fixtures re-org
* add xiami provider
* 整理代码:补充测试和文档
* 支持虾米音乐用户登录等功能
* 通过手动测试
@cosven
Copy link
Owner Author

cosven commented Aug 26, 2018

merged

@cosven cosven closed this Aug 26, 2018
@cosven cosven deleted the refactor-protocol branch August 26, 2018 14:49
@cosven cosven restored the refactor-protocol branch August 26, 2018 14:49
@cosven cosven deleted the refactor-protocol branch December 28, 2018 13:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants