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

Reverting the multi TFM update #634

Merged
merged 3 commits into from
May 9, 2024
Merged

Reverting the multi TFM update #634

merged 3 commits into from
May 9, 2024

Conversation

soninaren
Copy link
Member

No description provided.

@soninaren soninaren force-pushed the nasoni/installnet8 branch from d16407a to 5648e77 Compare May 8, 2024 22:17
@soninaren soninaren force-pushed the nasoni/installnet8 branch from c7c728f to ca74abf Compare May 8, 2024 23:24
@soninaren soninaren changed the title Installing net 8 sdk Reverting the multi TFM update May 8, 2024
Copy link
Contributor

@mattchenderson mattchenderson left a comment

Choose a reason for hiding this comment

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

Copy link
Member

@fabiocav fabiocav left a comment

Choose a reason for hiding this comment

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

To make things clear, you want to add the following to your project:

<RollForward>Major</RollForward>

This will enable the application to roll to the next major if 6.0 is not present.

For reference: https://learn.microsoft.com/en-us/dotnet/core/versions/selection#control-roll-forward-behavior

@soninaren soninaren force-pushed the nasoni/installnet8 branch from b113999 to 0b3bbe6 Compare May 9, 2024 00:51
@@ -87,7 +87,7 @@ WARNING: DO NOT MODIFY this file unless you are knowledgeable about MSBuild and
<Target Name="_InitializeFunctionsSdk">
<Warning Text="The ExtensionsMetadataGenerator package was not imported correctly. Are you missing '$(_ExtensionsMetadataGeneratorTargetsPath)' or '$(_ExtensionsMetadataGeneratorPropsPath)'?"
Condition="!$(AzureFunctionsVersion.StartsWith('v1')) And ('$(_ExtensionsMetadataGeneratorTargetsImported)' == '' Or '$(_ExtensionsMetadataGeneratorPropsImported)' == '')" />

Copy link
Member

Choose a reason for hiding this comment

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

nit: space

@soninaren soninaren requested a review from mattchenderson May 9, 2024 16:51
@soninaren soninaren dismissed mattchenderson’s stale review May 9, 2024 16:52

addressed feedback already

@soninaren soninaren merged commit 7881aea into v4.x May 9, 2024
2 checks passed
@soninaren soninaren deleted the nasoni/installnet8 branch May 9, 2024 16:53
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.

3 participants