Skip to content
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

(#2738) Uplift to .Net 4.8 #2739

Merged
merged 5 commits into from
Nov 7, 2022
Merged

Conversation

TheCakeIsNaOH
Copy link
Member

@TheCakeIsNaOH TheCakeIsNaOH commented Jun 16, 2022

Description Of Changes

TODO

Motivation and Context

TODO

Testing

TODO

Change Types Made

  • Bug fix (non-breaking change)
  • Feature / Enhancement (non-breaking change)
  • Breaking change (fix or feature that could cause existing functionality to change)

Related Issue

Fixes #2738

Change Checklist

  • Requires a change to the documentation
  • Documentation has been updated
  • Tests to cover my changes, have been added
  • All new and existing tests passed.
  • PowerShell v2 compatibility checked.

@TheCakeIsNaOH TheCakeIsNaOH force-pushed the uplift-framework branch 4 times, most recently from ea1562f to acd5517 Compare June 24, 2022 15:41
@coveralls
Copy link

coveralls commented Jun 24, 2022

Coverage Status

Coverage increased (+0.3%) to 27.839% when pulling 8487d6e on TheCakeIsNaOH:uplift-framework into d08d1f6 on chocolatey:develop.

@gep13 gep13 added this to the 2.0.0 milestone Jun 26, 2022
@TheCakeIsNaOH TheCakeIsNaOH force-pushed the uplift-framework branch 3 times, most recently from 907f0c7 to 065c4d6 Compare July 1, 2022 17:37
@TheCakeIsNaOH TheCakeIsNaOH force-pushed the uplift-framework branch 2 times, most recently from cae7191 to 7e8516f Compare July 20, 2022 13:57
@gep13 gep13 removed this from the 2.0.0 milestone Aug 4, 2022
@gep13 gep13 force-pushed the uplift-framework branch 2 times, most recently from e6a1dff to 3b2fb7c Compare November 1, 2022 09:06
@gep13 gep13 mentioned this pull request Nov 2, 2022
TheCakeIsNaOH and others added 5 commits November 7, 2022 09:56
These tests are broken by running on .Net 4.8, see issue chocolatey#2690
They are being skipped so the integration test run passes.
Now that it is building on a newer .Net version, it is not required to specify the
version explicitly, as it can be autodetected.
Updated the location of the system DLLs for the updated .NET version.
There has been a change in how GetCustomAttribute works between .NET
4.0 and .NET 4.8. Previously, this worked without any problems, but in
.NET 4.8, it errors with a message regarding duplicate types found. In
order to work around this, we make use of GetCustomAttributes and then
filter the list from there, as/when required.
@gep13 gep13 marked this pull request as ready for review November 7, 2022 10:47
Copy link
Member

@gep13 gep13 left a comment

Choose a reason for hiding this comment

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

LGTM!

@gep13
Copy link
Member

gep13 commented Nov 7, 2022

@TheCakeIsNaOH thank you very much for taking the time to do this work! This has been a long time coming, but we are ready to start making our way towards this goal! I have made a couple of changes to the files within this PR, as there were some changes required as a result of the work that was done in 1.2.0.

@gep13 gep13 merged commit d978181 into chocolatey:develop Nov 7, 2022
@TheCakeIsNaOH TheCakeIsNaOH deleted the uplift-framework branch November 17, 2022 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to target version 4.8 of the .NET Framework
3 participants