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.

Sign up for GitHub

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

Jump to bottom

Updating the version and removing old imports #432

Merged
merged 17 commits into from
Aug 26, 2024
Merged

Updating the version and removing old imports #432

merged 17 commits into from
Aug 26, 2024

Conversation

Paultagoras
Copy link
Contributor

Summary

Checklist

Delete items not relevant to your PR:

  • Unit and integration tests covering the common scenarios were added
  • A human-readable description of the changes was provided to include in CHANGELOG
  • For significant changes, documentation in https:/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

@Paultagoras Paultagoras self-assigned this Aug 25, 2024
@Paultagoras
Copy link
Contributor Author

Waiting on ClickHouse/clickhouse-java#1786

@mzitnik mzitnik self-requested a review August 26, 2024 06:23
@mzitnik
Copy link
Collaborator

mzitnik commented Aug 26, 2024

Almost all test are failing do you know why?

@Paultagoras
Copy link
Contributor Author

Almost all test are failing do you know why?

This is related to that bug I reported 🙁

@Paultagoras Paultagoras merged commit 671dea7 into main Aug 26, 2024
6 of 8 checks passed
@Paultagoras Paultagoras deleted the update-version branch August 26, 2024 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mzitnik mzitnik Awaiting requested review from mzitnik

Assignees

@Paultagoras Paultagoras

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Paultagoras @mzitnik