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.

Sign up for GitHub

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

Jump to bottom

Controller：topic 相关 #20

Closed
JacksonTian opened this issue Mar 5, 2018 · 7 comments
Closed

Controller：topic 相关 #20

JacksonTian opened this issue Mar 5, 2018 · 7 comments

Comments

@JacksonTian
Copy link
Member

No description provided.

@xiaomuzhu
Copy link
Contributor

我申请认领重构topic相关内容😄

@JacksonTian
Copy link
Member Author

Thanks @xiaomuzhu 1-3天内能完成吗？

@xiaomuzhu
Copy link
Contributor

@JacksonTian 可以的,不过是不是跟 #17 重复了?

@JacksonTian
Copy link
Member Author

@xiaomuzhu 有进展吗？

@xiaomuzhu
Copy link
Contributor

@JacksonTian 目前还差一个上传功能和测试部分.

有两个问题

  1. 老版本的upload功能分成了本地储存和七牛云储存,我们的逻辑是优先七牛云储存对吗?
    2.测试覆盖率有硬性要求吗?

@atian25
Copy link
Member

atian25 commented Mar 7, 2018

覆盖率我们期望是 100%

@JacksonTian
Copy link
Member Author

  1. 都要兼顾。
  2. 覆盖率可以分阶段达成吧。70%、90%、100%

@snyk-bot snyk-bot mentioned this issue Mar 24, 2020
Open
@snyk-bot snyk-bot mentioned this issue Dec 13, 2020
Open
This was referenced Mar 24, 2021
Open
Open
@snyk-bot snyk-bot mentioned this issue Sep 7, 2021
Open
@snyk-bot snyk-bot mentioned this issue Apr 7, 2022
Open
@snyk-bot snyk-bot mentioned this issue Aug 27, 2022
Open
@snyk-bot snyk-bot mentioned this issue Jan 10, 2023
Open
@atian25 atian25 mentioned this issue Jul 17, 2023
Open
@atian25 atian25 mentioned this issue Nov 14, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@atian25 @JacksonTian @xiaomuzhu