- Notifications
You must be signed in to change notification settings - Fork100
Update dotnet deps + run formatting on all relevant files + some fixes#1839
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
alerickson commentedJul 15, 2025
It looks like the build isn't working in CI for this PR |
o-l-a-v commentedJul 16, 2025
It builds fine on my Windows machine. 🤔 Seems GitHub actions builds on Ubuntu 22.04. Can it (Linux) even target net472? Why the Windows Server 2022 build fails I don't know. |
alerickson commentedJul 17, 2025
It builds fine on my (Windows) machine as well, but I just pulled in your most recent changes and it's (still) not building on the server. We only build on windows, and then we use that build to test Windows PowerShell on Windows and PowerShell Core on Windows, Mac and Ubuntu. Could you try opening up a new PR with a subset of these changes? It may help to have a smaller set of changes |
o-l-a-v commentedJul 18, 2025
Could we get more output on what/where it fails? |
Uh oh!
There was an error while loading.Please reload this page.
alerickson commentedJul 18, 2025
@adityapatwardhan's change resolved the build issue (just an incorrect file name), but it looks like many of the tests are failing with the same error: Error parsing server metadata: Could not load file or assembly |
o-l-a-v commentedAug 8, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Maybe previous run results are deleted, but the one failed test I can see does not fail when testing manually from Windows PowerShell. The name of the failed test was misleading though, Can this be rerun and can I get some logs to see where and what it fails on? |
alerickson commentedOct 3, 2025
I think this PR will need to be broken down to smaller PRs to see what's causing this issue:
|
o-l-a-v commentedOct 3, 2025
It's very slow to troubleshoot when tests don't run on PR automatically. 🫠 |
o-l-a-v commentedOct 3, 2025
Ithink it was due to upgrading |
alerickson commentedOct 3, 2025
Unfortunately now the package build is failing: |
o-l-a-v commentedOct 4, 2025
It builds fine on my machine, using dotnet v8.0.414. Would be helpful with more verbose build logs. If you don't want to continue with this PR just close it. Would be nice to have it working and merged, though. |
Uh oh!
There was an error while loading.Please reload this page.
This PR is just a suggestion on some housekeeping I think is due.
Feel free to close it or ask for changes.
PR Summary
dotnet-outdated.alexr00.formatallfilesinworkspace.Install-PSResourceof PSResourceGet in setupReleaseTools, not the aliasInstall-Module./vs\) in build scripts.PR Context
Why?
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.