Skip to content

fix: 补完 format code 遗漏的字段改名,修复 CI 编译失败 - #54

Merged
joesdu merged 1 commit into
mainfrom
dev
Oct 3, 2026
Merged

joesdu merged 1 commit into
mainfrom
dev

Conversation

@joesdu

@joesdu joesdu commented Oct 3, 2026

Copy link
Copy Markdown
Collaborator

修复 main 上 CI 的编译失败(run 37138490382)。

c797d28(format code)把一批后备字段收成了自动属性(含 field 关键字),但只改了声明所在的文件,同一 partial 类的其它文件仍在用旧字段名,Mongo / Redis / S3 插件编译失败(108 个错误)。

  • Mongo / Redis:其余 partial 文件里的旧字段名改成对应属性(_rows → Rows、_selected → SelectedItem、_lastRequest → CurrentRequest、_settings → Settings 等)。这些属性的 getter 都是 get;,读到的值与原字段相同。
  • GridFsTransfer.Bytes:驱动线程用 Interlocked.Exchange(ref …) 写它,属性取不了 ref,恢复 _bytes 字段。
  • S3 RemoveConfigAsync:各分支返回的响应类型不同,switch 表达式没有公共类型,还原为 switch 语句。
  • Redis 测试:两处 [ClassInitialize] 的形参 _ 是具名参数,方法体里新加的 _ = await … 成了给它赋值,形参改名 context。

本地验证

  • dotnet build VelaShell.Plugins.slnx:0 错误 0 警告
  • Telnet 26 / Serial 77 / DockerPanel 146 / S3 104 / Redis 155(不连服务器的部分)全过
  • Redis 集成与 UI 测试(garnet):82 过 2 败 —— 改动前的 258d05a 在 garnet 上同样这两个失败,是 garnet 的兼容性差异
  • Mongo(一次性容器,MongoDB 9.0.2):420 过 1 败 —— Explain_FindAndAggregateParse,在干净的 HEAD 上同样失败(9.0 改了 aggregate explain 的结构)

不改行为、接口与配置,velashell-docs 无需同步。

c797d28 把一批后备字段收成了自动属性(含 field 关键字),但只改了声明所在的文件,
同一 partial 类的其它文件仍在用旧字段名 —— Mongo / Redis / S3 插件编译失败(108 个错误)。

- Mongo / Redis:其余 partial 文件里的旧字段名改成对应属性(_rows → Rows、
  _selected → SelectedItem、_lastRequest → CurrentRequest、_settings → Settings 等);
  这些属性的 getter 都是 get;,读到的值与原字段相同
- GridFsTransfer.Bytes:驱动线程用 Interlocked.Exchange(ref …) 写它,属性取不了 ref,恢复 _bytes 字段
- S3 RemoveConfigAsync:各分支返回的响应类型不同,switch 表达式没有公共类型,还原为 switch 语句
- Redis 测试:两处 [ClassInitialize] 的形参 _ 是具名参数,方法体里新加的 `_ = await …`
  成了给它赋值,形参改名 context
@joesdu
joesdu merged commit 35773f6 into main Oct 3, 2026
1 check passed
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