Skip to content

feat: Add ClickHouse - #122

Merged
TethysPlex merged 9 commits into
linux-do:masterfrom
yyg-max:master
Sep 5, 2025
Merged

TethysPlex merged 9 commits into
linux-do:masterfrom
yyg-max:master

Conversation

@yyg-max

@yyg-max yyg-max commented Aug 30, 2025

Copy link
Copy Markdown
Collaborator

例行检查

  • 我已阅读并理解 贡献者公约,
  • 我已阅读并同意 贡献者许可协议 (CLA),确认我的贡献将根据项目的 MIT 许可证进行许可,
  • 我知晓如果此 PR 并不做出实质性更改,或可被认为是为了PR被合并而提交PR的,则可能不会被合并,

变更内容

Add ClickHouse

@TethysPlex

Copy link
Copy Markdown
Contributor

我怎么感觉这不像只改了 ClickHouse 呢()

@yyg-max

yyg-max commented Aug 30, 2025 •

Copy link
Copy Markdown
Collaborator Author

我怎么感觉这不像只改了 ClickHouse 呢()

只加了ClickHouse ,改了分数没达标的提示

TethysPlex
TethysPlex previously approved these changes Sep 1, 2025

@TethysPlex TethysPlex left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Expected indentation of 10 spaces but found 8.
'ProjectData' is defined but never used.
@yyg-max
yyg-max requested a review from TethysPlex September 1, 2025 06:49

@TethysPlex TethysPlex left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@TethysPlex
TethysPlex merged commit 151e259 into linux-do:master Sep 5, 2025
9 checks 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.

2 participants