-
Notifications
You must be signed in to change notification settings - Fork 640
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 petercat #2660
base: master
Are you sure you want to change the base?
Feat petercat #2660
Conversation
🦋 Changeset detectedLatest commit: 8ed9d03 The changes in this PR will be included in the next version bump. This PR includes changesets to release 12 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
WalkthroughThis pull request introduces several updates and improvements across multiple packages in the repository. It includes version updates for dependencies, changes in import paths, and the removal of unused code. Additionally, there are updates related to SEO components and the handling of raster tile extents. Changes
|
@@ -241,9 +241,11 @@ export default class FontService extends EventEmitter implements IFontService { | |||
|
|||
// 3. layout characters | |||
if (sdf) { | |||
// @ts-ignore |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using @ts-ignore
should be avoided unless absolutely necessary, as it suppresses TypeScript's type checking. Consider addressing the underlying type issues instead.
[English Template / 英文模板]
🤔 这个变动的性质是?
🔗 相关 Issue
💡 需求背景和解决方案
📝 更新日志
☑️ 请求合并前的自查清单