-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Updated dotnet version to 8.0.101 #15475
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
Conversation
This is a corresponding PR to https://github.com/github/semmle-code/pull/48786
@henti I pushed an extra commit to this PR to fix the failing integration tests. Let's see the results of the new CI run... |
There are some |
Yeah, I think we need to bump QL submodule on the internal PR to verify that the tests will pass when the PRs are merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
I've merged #15472, which adds further integration tests that depend on 8.0.0. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
This is a corresponding PR to https://github.com/github/semmle-code/pull/48786