Skip to content

[BUGFIX] Add lint command in ClickHouse and fix linter errors#547

Merged
jgbernalp merged 2 commits intoperses:mainfrom
opnpulse:clickhouse-linter-fix
Feb 4, 2026
Merged

[BUGFIX] Add lint command in ClickHouse and fix linter errors#547
jgbernalp merged 2 commits intoperses:mainfrom
opnpulse:clickhouse-linter-fix

Conversation

@rafi-ruetcse17
Copy link
Contributor

Description

Lint fixes

This change fixes the remaining ESLint issues introduced during the previous update in #241.
It resolves import order violations, replaces any with explicit TypeScript types, removes unused definitions, and applies required formatting fixes so the ClickHouse plugin passes lint checks.

Checklist

  • Pull request has a descriptive title and context useful to a reviewer.
  • Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the
    following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, DOC,IGNORE.
  • All commits have DCO signoffs.

Signed-off-by: rafi <rafialam@appscode.com>
@rafi-ruetcse17 rafi-ruetcse17 requested a review from a team as a code owner February 2, 2026 05:05
@rafi-ruetcse17 rafi-ruetcse17 requested review from jgbernalp and removed request for a team February 2, 2026 05:05
@jgbernalp
Copy link
Contributor

@rafi-ruetcse17 it seems there still some minor type issues

Signed-off-by: rafi <rafialam@appscode.com>
@rafi-ruetcse17
Copy link
Contributor Author

@rafi-ruetcse17 it seems there still some minor type issues

updated

@jgbernalp jgbernalp added this pull request to the merge queue Feb 4, 2026
Merged via the queue into perses:main with commit d4de825 Feb 4, 2026
13 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