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.

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

Updated dependencies and Improvements to CI process #188

Closed
wants to merge 1 commit into from
Closed

Updated dependencies and Improvements to CI process #188

wants to merge 1 commit into from

Conversation

gaby
Copy link

@gaby gaby commented Apr 30, 2023

  • Updated all dependencies to their latest version.
  • Removed samara-cluster since it's deprecated and merged into Shopify/sarama. The code for generating this was done using GPT-4
  • Updated go.mod file with go 1.19.
  • Ran go fmt on the code base.
  • Added support for tracking dependencies using dependabot.
  • Added support for running the tests with multiple Go versions and multiple OS platforms.
  • Added support for building Docker image using Github Actions. The image will be published to ghcr.io which may need to be enable for this repo/org.
  • Updated the image tag in the Dockerfile.

Note:

  • GPT4 suggested changing the logic of ingestClickHouse() to improve performance.

The following can be done in a separate PR:

  • Add support for golangci-lint, and fix the issues
  • Add support for gosec and fix the issues
  • Cleanup the Makefile

Related to #172
Fixes #187

Signed-off-by: Juan Calderon-Perez <jgcalderonperez@protonmail.com>
@gaby gaby mentioned this pull request Apr 30, 2023
@gaby gaby closed this Feb 26, 2024
@gaby gaby deleted the updates branch February 26, 2024 06:03
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