Skip to content

chore(deps): Update .NET version for .NET client smoke test ##633

Merged
grayside merged 2 commits intomainfrom
test-dotnet-8
Jun 6, 2025
Merged

chore(deps): Update .NET version for .NET client smoke test ##633
grayside merged 2 commits intomainfrom
test-dotnet-8

Conversation

@grayside
Copy link
Contributor

@grayside grayside commented Jun 6, 2025

This is a .NET equivalent of a problem similar to #632. Depending on the order in which broken checks are resolved, this may also require an admin merge to bypass required checks.

According to the logs for a failing smoke test at https://github.com/googleapis/google-cloudevents/actions/runs/14789546588/job/41523929026,

Install the [8.0.408] .NET SDK or update [/home/runner/work/google-cloudevents/google-cloudevents/google-cloudevents-dotnet/global.json] to match an installed SDK.

I am not familiar with .NET runtimes but trying a naive update of the number to see what happens.

@product-auto-label product-auto-label bot added the api: eventarc Issues related to the googleapis/google-cloudevents API. label Jun 6, 2025
@grayside grayside marked this pull request as ready for review June 6, 2025 22:44
@grayside grayside requested a review from a team as a code owner June 6, 2025 22:44
@grayside
Copy link
Contributor Author

grayside commented Jun 6, 2025

Would be great to get more feedback from @jskeet, but with a passing test I think it's worth moving forward with this change.

@grayside grayside self-assigned this Jun 6, 2025
Copy link

@iennae iennae left a comment

Choose a reason for hiding this comment

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

this looks good to me.

@grayside grayside merged commit f1afc28 into main Jun 6, 2025
10 of 12 checks passed
@grayside grayside deleted the test-dotnet-8 branch June 6, 2025 23:54
@jskeet
Copy link
Collaborator

jskeet commented Jun 9, 2025

Yup, this is the right fix - thanks for making it, and apologies for the inconvenience. (googleapis/google-cloudevents-dotnet#319 is where we changed to use .NET 8.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: eventarc Issues related to the googleapis/google-cloudevents API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants