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

add embed files to the package chaincode #4729

Merged
merged 1 commit into from
Mar 7, 2024
Merged

Conversation

pfi79
Copy link
Contributor

@pfi79 pfi79 commented Mar 7, 2024

Recently pr failed tests and failed to update the google.golang.org/protobuf package.
The reason is that we added go:embed to the https://github.com/protocolbuffers/protobuf-go/blob/master/internal/editiondefaults/defaults.go. When building the package we didn't copy file editions_defaults.binpb and the build of the chaincode ended with an error.

After accepting this pr, you can update the package again

Signed-off-by: Fedor Partanskiy <pfi79@mail.ru>
@pfi79 pfi79 requested a review from a team as a code owner March 7, 2024 07:39
@pfi79
Copy link
Contributor Author

pfi79 commented Mar 7, 2024

@denyeart

@C0rWin C0rWin merged commit 5ba69f6 into hyperledger:main Mar 7, 2024
14 checks passed
@pfi79 pfi79 deleted the embed-files branch March 7, 2024 11:36
@denyeart
Copy link
Contributor

denyeart commented Mar 7, 2024

@Mergifyio backport release-2.5

Copy link

mergify bot commented Mar 7, 2024

backport release-2.5

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Mar 7, 2024
Signed-off-by: Fedor Partanskiy <pfi79@mail.ru>
(cherry picked from commit 5ba69f6)
denyeart pushed a commit that referenced this pull request Mar 7, 2024
Signed-off-by: Fedor Partanskiy <pfi79@mail.ru>
(cherry picked from commit 5ba69f6)
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.

None yet

3 participants