[go: up one dir, main page]

proto: use operation encoding with attestation name by default

Albin Coquereaurequested to merge
albin@use-attestation-encoding-in-protocol into master

What

This MR unrevert !10422 (merged) and made the attestation name the default one for consensus operations.

Why

This is part of the renaming of endorsement project. With the acceptation of Protocol Oxford by the community, we can go to the next step for this renaming. In the next protocol, P, attestation should be the default name for consensus operation.

How

This MR is mainly the cherry picking of commits from !9827 (merged) (with some adaptation to the current protocol code) + a commit that change the encoding use in the lib_injector.

Manually testing the MR

CI Green

Checklist

  • Document the interface of any function added or modified (see the coding guidelines)
  • Document any change to the user interface, including configuration parameters (see node configuration)
  • Provide automatic testing (see the testing guide).
  • For new features and bug fixes, add an item in the appropriate changelog (docs/protocols/alpha.rst for the protocol and the environment, CHANGES.rst at the root of the repository for everything else).
  • Select suitable reviewers using the Reviewers field below.
  • Select as Assignee the next person who should take action on that MR

Merge request reports