Skip to content

Roll back to .NET 6 #3171

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

Merged
merged 2 commits into from
Feb 24, 2025
Merged

Roll back to .NET 6 #3171

merged 2 commits into from
Feb 24, 2025

Conversation

timayabi2020
Copy link
Contributor

This PR rolls back the authentication module to .NET 6 due to the breaking change in the .NET SDK version 8.x. which is not compatible with Azure Automation Run Book's PowerShell 7.2 version.

@timayabi2020 timayabi2020 marked this pull request as ready for review February 24, 2025 16:18
@timayabi2020 timayabi2020 requested a review from a team as a code owner February 24, 2025 16:18
@baywet
Copy link
Member

baywet commented Feb 24, 2025

For context this is a manual rollback of #3078

@timayabi2020 I'm not sure whether you're aware of the git revert command, extremely useful in those cases :)

@timayabi2020 timayabi2020 merged commit b4a8458 into dev Feb 24, 2025
3 of 5 checks passed
@timayabi2020 timayabi2020 deleted the net-6-rollback branch February 24, 2025 17:04
@nkasco
Copy link

nkasco commented Feb 26, 2025

This type of change, while warranted, really needs to be better documented as a breaking change so that users have full awareness without digging.

So am I interpreting correctly that because another MS product is not up to date, that this change was rolled back and the SDK will continue to work on 7.2 and 7.3?

@o-l-a-v
Copy link

o-l-a-v commented Feb 26, 2025

Actually, Automation Account supports PowerShell 7.4, has done since november 2024. It's just not documented anywhere, and thus probably not GA?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants