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

Commons #1

Merged
merged 10 commits into from
Dec 5, 2023
Merged

Commons #1

merged 10 commits into from
Dec 5, 2023

Conversation

lovromazgon
Copy link
Member

@lovromazgon lovromazgon commented Nov 30, 2023

This PR ports code from Conduit and the Conduit Connector SDK into this module so they can be shared in the future between both of them and the Processor SDK which will be introduced in the short future.

I want to call out that the golangci-lint action is now configured to take the version from go.mod, we should probably introduce this to other projects as well (especially Conduit). This means that dependabot updates the version used in the CI action and we don't have to do anything, unless the linter action fails. (EDIT: here it is ConduitIO/conduit#1300)

Closes ConduitIO/conduit#1284

@lovromazgon lovromazgon requested a review from a team November 30, 2023 18:41
.github/dependabot.yml Outdated Show resolved Hide resolved
opencdc/data.go Show resolved Hide resolved
opencdc/position.go Outdated Show resolved Hide resolved
opencdc/record.go Outdated Show resolved Hide resolved
opencdc/record_test.go Show resolved Hide resolved
opencdc/record.go Outdated Show resolved Hide resolved
- change github actions dependabot interval to weekly
- use goccy/go-json instead of encoding/json
- fix comments
- add Record.WithSerializer
- add benchmark for Record.Clone
@lovromazgon lovromazgon requested a review from hariso December 5, 2023 17:52
Copy link
Contributor

@hariso hariso left a comment

Choose a reason for hiding this comment

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

🚀

@lovromazgon lovromazgon merged commit e5f0f24 into main Dec 5, 2023
4 checks passed
@lovromazgon lovromazgon deleted the lovro/commons branch December 5, 2023 18:07
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.

BP: Extract common code
2 participants