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

Rewrite component storage #1889

Merged
merged 6 commits into from
Feb 24, 2022
Merged

Rewrite component storage #1889

merged 6 commits into from
Feb 24, 2022

Conversation

Him188
Copy link
Member

@Him188 Him188 commented Feb 17, 2022

See commits for changes

@Him188 Him188 added s:console 子系统: mirai-console x:behavior-change 属性: 行为变更 (API 不变, 行为改变) labels Feb 17, 2022
@Him188 Him188 added this to the 2.11.0-RC milestone Feb 17, 2022
@Him188 Him188 force-pushed the rewrite-compoennt-storage branch from 327d6b7 to 3d888e1 Compare February 17, 2022 19:22
@Him188 Him188 marked this pull request as ready for review February 17, 2022 22:48
Copy link
Member

@Karlatemp Karlatemp left a comment

Choose a reason for hiding this comment

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

PermissionServiceProvider 是有个阻止超前调用的逻辑的, 这个 pr 似乎把这个强要求给去除了

@Him188 Him188 changed the title Rewrite compoennt storage Rewrite component storage Feb 19, 2022
@Him188 Him188 added x:deprecation 属性: API 弃用 x:abi-change 属性: ABI 变更 and removed x:behavior-change 属性: 行为变更 (API 不变, 行为改变) labels Feb 19, 2022
… `priority`. Remove `builtinImplementations` and contribute them at the first initialization phase instead.

Close #1888, fix #1860.

Add `ComponentStorageInternal` for frontend to provide components.

Deprecate:
- SingletonExtension
- SingletonExtensionPoint
- AbstractSingletonExtensionPoint
- SingletonExtensionSelector
- CommandCallInterceptorProviderImpl
- CommandCallInterceptorProviderImplLazy
- CommandCallParserProviderImpl
- CommandCallParserProviderImplLazy
- CommandCallResolverProviderImpl
- CommandCallResolverProviderImplLazy

ABI breaking change:
- `PermissionServiceProvider`: supertype changed
- `CommandCallResolverProvider.ExtensionPoint`: supertype changed
- `PermissionServiceProvider.ExtensionPoint`: supertype changed
@Him188 Him188 force-pushed the rewrite-compoennt-storage branch from 3d888e1 to defdf30 Compare February 22, 2022 19:38
@Him188 Him188 requested review from Karlatemp and removed request for Karlatemp February 22, 2022 19:38
@Him188 Him188 force-pushed the rewrite-compoennt-storage branch from defdf30 to 1ada8d5 Compare February 22, 2022 19:53
@Him188 Him188 requested a review from Karlatemp February 22, 2022 19:54
@Him188
Copy link
Member Author

Him188 commented Feb 22, 2022

确实, 添加了一个 commit, force 是为了 rebase onto dev

@Him188 Him188 added the z:ready-to-merge 状态: PR 已经可以合并, 正在等待一些安排 label Feb 24, 2022
@Him188 Him188 merged commit 77e2a5c into dev Feb 24, 2022
@Him188 Him188 deleted the rewrite-compoennt-storage branch February 24, 2022 11:05
@Him188 Him188 removed this from the 2.11.0-RC milestone Mar 8, 2022
@Him188 Him188 added this to the 2.11.0-M1 milestone Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s:console 子系统: mirai-console x:abi-change 属性: ABI 变更 x:deprecation 属性: API 弃用 z:ready-to-merge 状态: PR 已经可以合并, 正在等待一些安排
Projects
None yet
Development

Successfully merging this pull request may close these issues.

优化 ComponentStorage 搜索策略 contributeCommandCallParser() 对CommandCallParser的扩展不运作
2 participants