Skip to content

refactor!: migrate to context parameters#13

Closed
Aliorpse wants to merge 5 commits intomainfrom
feat/context-parameters
Closed

refactor!: migrate to context parameters#13
Aliorpse wants to merge 5 commits intomainfrom
feat/context-parameters

Conversation

@Aliorpse
Copy link
Copy Markdown
Contributor

@Aliorpse Aliorpse commented Apr 3, 2026

迁移到 Conext Parameters,将在其稳定后合并。本 PR 有大量破坏性改动。

改动:

  • 部分扩展 API 从依赖接收者迁移到依赖上下文参数(XxxEnvironment),如 on command regex respond,使在不同的定义环境下调用的是同一个函数,减少代码量。
  • SaltifyCommandRequirmentContext 重命名为 CommandRequirementMatchRegisteredCommandInfo 重命名为 RegisteredCommand

丞待解决的项:

  • 更新文档

@Aliorpse Aliorpse force-pushed the main branch 4 times, most recently from 35c37ab to 4e02ab0 Compare April 5, 2026 04:26
Aliorpse and others added 3 commits April 5, 2026 12:37
# Conflicts:
#	saltify-core/src/commonMain/kotlin/org/ntqqrev/saltify/dsl/SaltifyCommandContext.kt
#	saltify-core/src/commonMain/kotlin/org/ntqqrev/saltify/extension/ApplicationExtension.kt
#	saltify-core/src/commonMain/kotlin/org/ntqqrev/saltify/extension/CommandContextExtension.kt
#	saltify-core/src/jvmTest/kotlin/org/ntqqrev/saltify/PluginTest.kt
@Aliorpse Aliorpse closed this Apr 11, 2026
@Aliorpse Aliorpse deleted the feat/context-parameters branch April 11, 2026 15:42
@Aliorpse
Copy link
Copy Markdown
Contributor Author

这个分支所实现的特性不用 Context Parameter 同样可以做到,故关闭

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.

1 participant