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: update vutils/vscale version to support custom callback in tick… #1378

Merged
merged 3 commits into from
Aug 15, 2024

Conversation

666haiwen
Copy link
Contributor

@666haiwen 666haiwen commented Aug 15, 2024

Add function callback options in tickmode to support custom ticks such as wilkson etc.
Can use customTicks with function in @visactor/vscale such as wilkinsonExtended function.(Related PR:VisActor/VUtil#198)

  • New feature
  • Bug fix
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Refactoring
  • Update dependency
  • Code style optimization
  • Test Case
  • Branch merge
  • Site / documentation update
  • Demo update
  • Workflow
  • Release
  • Other (about what?)

🔗 Related issue link

🐞 Bugserver case id

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English tickmode support custom function callback
🇨🇳 Chinese tickmode配置新增自定义刻度的配置

☑️ Self-Check before Merge

⚠️ Please check all items below before requesting a reviewing. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

🚀 Summary

copilot:summary

🔍 Walkthrough

copilot:walkthrough

@666haiwen 666haiwen requested a review from xile611 August 15, 2024 03:38
import type { Point, TextContent } from '../core/type';
import type { SegmentAttributes } from '../segment';
import type { TagAttributes } from '../tag';
import type { BaseScale } from '@visactor/vscale/es/base-scale';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个引用不对

@@ -75,5 +75,6 @@
},
"publishConfig": {
"access": "public"
}
},
"packageManager": "pnpm@9.4.0+sha512.f549b8a52c9d2b8536762f99c0722205efc5af913e77835dbccc3b0b0b2ca9e7dc8022b78062c17291c48e88749c70ce88eb5a74f1fa8c4bf5e18bb46c8bd83a"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个是不是因为直接用了pnpm 安装依赖,我们现在是 rush update 更新依赖
rush底层用的也是pnpm

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

嗯,跑jest的时候用了pnpm命令导致的

@xile611 xile611 added this to the 0.20.0 milestone Aug 15, 2024
@xile611 xile611 merged commit d7161ab into VisActor:dev/0.20.0 Aug 15, 2024
3 of 5 checks passed
@666haiwen 666haiwen mentioned this pull request Aug 16, 2024
20 tasks
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.

2 participants