Allow a connector's or processor's plugin name to be updated #1938

hariso · 2024-11-01T13:59:22Z

Fixes #1929.

I have followed the Code Guidelines.
There is no other pull request for the same update/change.
I have written unit tests.
I have made sure that the PR is of reasonable size and can be easily reviewed. -- Most changes are boring propagations of the plugin argument.

github-actions · 2024-11-01T13:59:42Z

The latest Buf updates on your PR. Results from workflow buf-validate / validate (pull_request).

Build	Format	Lint	Breaking	Updated (UTC)
`✅ passed`	`✅ passed`	`✅ passed`	`✅ passed`	Nov 8, 2024, 2:30 PM

raulb

Went through this pull-request with @hariso and made perfect sense to me. Great work.

proto/buf.yaml

hariso added 3 commits November 1, 2024 13:12

allow plugin change

40fde5b

tests

88c875d

update tests

36a07dc

tests

90e1733

hariso marked this pull request as ready for review November 1, 2024 17:34

hariso requested a review from a team as a code owner November 1, 2024 17:34

hariso added 2 commits November 4, 2024 12:38

log

a47a9ee

update processors

3bc7d2d

hariso changed the title ~~Allow a connector's plugin name to be updated~~ Allow a connector's or processor's plugin name to be updated Nov 4, 2024

hariso added 2 commits November 4, 2024 15:02

fix tests

f0dde31

update test

1068050

raulb approved these changes Nov 8, 2024

View reviewed changes

proto/buf.yaml Show resolved Hide resolved

Merge branch 'main' into haris/relaxed-plugins

e06c403

raulb enabled auto-merge (squash) November 8, 2024 14:31

raulb merged commit 977b037 into main Nov 8, 2024
4 checks passed

raulb deleted the haris/relaxed-plugins branch November 8, 2024 15:06

Provide feedback