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

refactor: remove unused path-util functions #82

Merged
merged 5 commits into from
Jun 19, 2022
Merged

refactor: remove unused path-util functions #82

merged 5 commits into from
Jun 19, 2022

Conversation

xiaoiver
Copy link
Contributor

@xiaoiver xiaoiver commented Jun 17, 2022

#81

重构 path-util:

  • 移除部分未使用的方法
  • @antv/g 中使用的 path 相关方法移入,例如用于 morph 动画的 cubic curve 分割
  • 修复部分命令(S)转曲问题

使用文档已更新 README.md

@lgtm-com
Copy link

lgtm-com bot commented Jun 17, 2022

This pull request fixes 4 alerts when merging 15d00ae into 6ff61cc - view on LGTM.com

fixed alerts:

  • 2 for Useless assignment to local variable
  • 2 for Useless conditional

@lgtm-com
Copy link

lgtm-com bot commented Jun 18, 2022

This pull request fixes 4 alerts when merging f028741 into 6ff61cc - view on LGTM.com

fixed alerts:

  • 2 for Useless assignment to local variable
  • 2 for Useless conditional

@xiaoiver xiaoiver self-assigned this Jun 19, 2022
@lgtm-com
Copy link

lgtm-com bot commented Jun 19, 2022

This pull request fixes 4 alerts when merging 594d9aa into 6ff61cc - view on LGTM.com

fixed alerts:

  • 2 for Useless assignment to local variable
  • 2 for Useless conditional

@lgtm-com
Copy link

lgtm-com bot commented Jun 19, 2022

This pull request fixes 4 alerts when merging a273ca2 into 6ff61cc - view on LGTM.com

fixed alerts:

  • 2 for Useless assignment to local variable
  • 2 for Useless conditional

@xiaoiver xiaoiver merged commit cc2d9cc into master Jun 19, 2022
@xiaoiver xiaoiver deleted the fix-path branch June 19, 2022 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant