-
Notifications
You must be signed in to change notification settings - Fork 255
updates for .NET 9 #3142
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
updates for .NET 9 #3142
Changes from 14 commits
32abdb7
2dc10a0
e6b0e7a
533e6a5
fd590e6
aca9b39
c17c8e4
bfe0ebe
08d803f
7f67603
88be0e2
49fb8bd
f48775d
162e3af
0e4abe9
ba658f3
7a315fc
11eeee1
0ecc61b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -16,7 +16,7 @@ foreach ($line in $($publishOutput -split "`r`n")) | |
| } | ||
|
|
||
| Write-Host "Actual warning count is: ", $actualWarningCount | ||
| $expectedWarningCount = 63 | ||
| $expectedWarningCount = 61 | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. if the count is lower, that is okay. we don't want it going up. I assume some dependencies were updated.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are you saying that 2 APIs were made AOT compatible so they don't throw warnings? Should we find and compare which ones they are? |
||
|
|
||
| if ($LastExitCode -ne 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.
this seems to help against any issues w/the frameworks not running properly, there seemed to be an issue w/the wrong framework running on the OIDC tests
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.
I tried installing all frameworks yesterday, it didn't help. But if this works, then it works...
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.
it worked when I did a test PR
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.
The run failed?
https://github.com/AzureAD/microsoft-identity-web/actions/runs/11822912264
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.
That's not what it showed. and then I changed the base branch so I don't have proof.