Skip to content

Conversation

@zcsizmadia
Copy link
Contributor

What is the purpose of the change

Installing .NET 7 SDK in the docker image, fixing AVRO-3738.)

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

@github-actions github-actions bot added the build label Apr 19, 2023
@zcsizmadia
Copy link
Contributor Author

zcsizmadia commented Apr 19, 2023

@RyanSkraba This might fix your problem. I assume you use the Dockerfile to build and run the tests. The github workflows do not use this dockerfile, and they work as expected, and unfortunately I forgot about that Dockerfile when made the original PR

@zcsizmadia zcsizmadia requested a review from RyanSkraba April 19, 2023 16:08
@github-actions github-actions bot added the C# label Apr 19, 2023
@zcsizmadia zcsizmadia self-assigned this Apr 19, 2023
Copy link
Contributor

@RyanSkraba RyanSkraba left a comment

Choose a reason for hiding this comment

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

Thanks so much for the rapid fix!

@RyanSkraba RyanSkraba merged commit 6f0692f into apache:master Apr 20, 2023
@zcsizmadia zcsizmadia deleted the avro-3738-add-dotnet-install-7-to-dockerfile branch April 20, 2023 16:25
RyanSkraba pushed a commit to RyanSkraba/avro that referenced this pull request Jun 13, 2023
* Add NET 7 SDK to Dockerfile

* Update code analyzer package

---------

Co-authored-by: Zoltan Csizmadia <[email protected]>
RyanSkraba pushed a commit that referenced this pull request Jun 14, 2023
* Add NET 7 SDK to Dockerfile

* Update code analyzer package

---------

Co-authored-by: Zoltan Csizmadia <[email protected]>
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.

2 participants