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

Use built-in GetRequiredFeature when available #512

Merged
merged 7 commits into from
Aug 5, 2024
Merged

Conversation

twsouthwick
Copy link
Member

This change updates to use the in-box GetRequiredFeature. As part of this, the .NET 7 target is dropped for v2.0 since it's out of support and made building this tricky.

This change updates to use the in-box GetRequiredFeature. As part of this, the .NET 7 target is dropped for v2.0 since it's out of support and made building this tricky.
@twsouthwick twsouthwick added this to the 2.0 milestone Jul 23, 2024
@twsouthwick twsouthwick enabled auto-merge (squash) July 23, 2024 23:16
Copy link
Member

@mjrousos mjrousos left a comment

Choose a reason for hiding this comment

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

LGTM

@twsouthwick twsouthwick disabled auto-merge August 5, 2024 15:47
@twsouthwick twsouthwick enabled auto-merge (squash) August 5, 2024 15:50
@twsouthwick twsouthwick merged commit 5bbe34b into main Aug 5, 2024
5 checks passed
@twsouthwick twsouthwick deleted the required-feature branch August 5, 2024 18:00
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.

2 participants