Skip to content

Conversation

@ChrisPulman
Copy link
Member

Removed the obsolete FilterOnProperty extension and related tests.
Updated .NET workflow to support .NET 10 and improved caching.
Upgraded package dependencies in Directory.Build.props and test project.
Updated copyright years.

Removed the obsolete FilterOnProperty extension and related tests. Updated .NET workflow to support .NET 10 and improved caching. Upgraded package dependencies in Directory.Build.props and test project. Updated copyright years.
Copy link
Collaborator

@JakenVeina JakenVeina left a comment

Choose a reason for hiding this comment

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

We'll need a major version bump for the removal of obsolete APIs, I think, in which case there's at least one other outstanding PR that's been waiting for a major version bump. We should make sure and pull it before releasing.

public static class ObsoleteEx
{
}
public static class ObsoleteEx;
Copy link
Collaborator

Choose a reason for hiding this comment

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

If we're making breaking changes to remove obsolete methods anyway, should we remove this too?

9.0.x
10.0.x
dotnet-quality: 'preview'
Copy link
Collaborator

Choose a reason for hiding this comment

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

It looks like the indentation is off here, and is causing the build to fail. dotnet-quality, cache, and cache-dependency-path I should all be indented to the same level as dotnet-version.

Copy link
Member

Choose a reason for hiding this comment

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

Don't need the dotnet-quality of preview now that .net 10 is out anwyay.

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