Skip to content

Remove protobuf dependencies#609

Merged
thehoul merged 16 commits into
adapt-protobuf-testfrom
remove-protobuf-dependencies
Feb 27, 2026
Merged

Remove protobuf dependencies#609
thehoul merged 16 commits into
adapt-protobuf-testfrom
remove-protobuf-dependencies

Conversation

@thehoul

@thehoul thehoul commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

Follow up to #611 where protobuf is moved to internal. This PR changes all imports to use only internal protobuf

@thehoul thehoul self-assigned this Feb 3, 2026
@github-actions

github-actions Bot commented Feb 3, 2026

Copy link
Copy Markdown

🔒 Could not start CI tests due to missing safe PR label. Please contact a DEDIS maintainer.

@thehoul thehoul mentioned this pull request Feb 3, 2026
@thehoul
thehoul force-pushed the move-protobuf-to-internal branch from d0891ba to 7d486ae Compare February 5, 2026 10:37
@thehoul
thehoul changed the base branch from move-protobuf-to-internal to adapt-protobuf-test February 5, 2026 10:46
@thehoul
thehoul force-pushed the remove-protobuf-dependencies branch from 1fdaf5f to 562ac70 Compare February 5, 2026 10:52
@github-actions

github-actions Bot commented Feb 5, 2026

Copy link
Copy Markdown

🔒 Could not start CI tests due to missing safe PR label. Please contact a DEDIS maintainer.

@thehoul thehoul added the safe PR label Feb 5, 2026
@thehoul
thehoul marked this pull request as ready for review February 5, 2026 10:54

@AnomalRoil AnomalRoil left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@AnomalRoil AnomalRoil left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Comment thread internal/v3marshaling.go Outdated
Co-authored-by: AnomalRoil <AnomalRoil@users.noreply.github.com>
@sonarqubecloud

Copy link
Copy Markdown

@thehoul
thehoul merged commit 78c552f into adapt-protobuf-test Feb 27, 2026
20 of 32 checks passed
@thehoul
thehoul deleted the remove-protobuf-dependencies branch February 27, 2026 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants