Skip to content

Bugfix/#77 review protobuf merge from usage to avoid list entry duplication#78

Merged
pLeminoq merged 7 commits intofeature/testsfrom
bugfix/#77_review_protobuf_merge_from_usage_to_avoid_list_entry_dublication
Nov 3, 2022
Merged

Bugfix/#77 review protobuf merge from usage to avoid list entry duplication#78
pLeminoq merged 7 commits intofeature/testsfrom
bugfix/#77_review_protobuf_merge_from_usage_to_avoid_list_entry_dublication

Conversation

@DivineThreepwood
Copy link
Copy Markdown
Member

@DivineThreepwood DivineThreepwood commented Oct 27, 2022

Based on: openbase/jul#

📜 Description

  • removemergeForm usage in ActionImpl
  • replace all mergeFrom usages by mergeFromWithoutRepeatedFields that is now provided by jul.

@pLeminoq pLeminoq merged commit 5a4a850 into feature/tests Nov 3, 2022
@pLeminoq pLeminoq deleted the bugfix/#77_review_protobuf_merge_from_usage_to_avoid_list_entry_dublication branch November 3, 2022 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review Usage of mergeFrom on Protobuf Types

2 participants