Skip to content
This repository was archived by the owner on Oct 7, 2023. It is now read-only.

Conversation

Aarebecca
Copy link
Contributor

  • 完善单测
  • 调整了代码结构,将parseMarker移至util
  • 修改了案例
  • 具有update方法

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1032601589

  • 19 of 20 (95.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.6%) to 80.558%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/ui/marker/utils.ts 13 14 92.86%
Totals Coverage Status
Change from base Build 1029448404: 0.6%
Covered Lines: 739
Relevant Lines: 878

💛 - Coveralls

@visiky
Copy link
Member

visiky commented Jul 18, 2021

@hustcc 单测文件命名有规范吗?我觉得可以同 G2Plot 统一,以及 G2 5.0 后续也保持 xxx-spec.ts 的格式

@visiky
Copy link
Member

visiky commented Jul 18, 2021

考虑改一下 单测文件命名

@Aarebecca
Copy link
Contributor Author

考虑改一下 单测文件命名

好的

@hustcc
Copy link
Member

hustcc commented Jul 19, 2021

冲突了,git rebase 一下 master。

@visiky visiky merged commit 2a6f6cb into antvis:master Jul 23, 2021
@Aarebecca Aarebecca deleted the refactor-marker branch July 23, 2021 05:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants