Skip to content

.Net: Remove obsoleted MEVD APIs#11891

Merged
roji merged 1 commit intomicrosoft:feature-vector-data-preb3from
roji:LastObsoletes
May 5, 2025
Merged

.Net: Remove obsoleted MEVD APIs#11891
roji merged 1 commit intomicrosoft:feature-vector-data-preb3from
roji:LastObsoletes

Conversation

@roji
Copy link
Member

@roji roji commented May 5, 2025

This removes all MEVD obsolete APIs except for the legacy filters, which are still required by SK's ITextSearch; this is tracked by #10456.

Closes #10809

@markwallace-microsoft markwallace-microsoft added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel memory labels May 5, 2025
@roji roji temporarily deployed to integration May 5, 2025 15:33 — with GitHub Actions Inactive
@github-actions github-actions bot changed the title Remove obsoleted MEVD APIs .Net: Remove obsoleted MEVD APIs May 5, 2025
@roji roji marked this pull request as ready for review May 5, 2025 15:44
@roji roji requested a review from a team as a code owner May 5, 2025 15:44
@roji roji force-pushed the LastObsoletes branch from c992f93 to bbdc568 Compare May 5, 2025 17:04
@roji roji temporarily deployed to integration May 5, 2025 17:04 — with GitHub Actions Inactive
@roji roji merged commit 8e4d993 into microsoft:feature-vector-data-preb3 May 5, 2025
11 checks passed
@roji roji deleted the LastObsoletes branch May 5, 2025 17:28
Copy link
Member

@adamsitnik adamsitnik left a comment

Choose a reason for hiding this comment

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

I sleep better after this PR gets merged :D

image

@roji
Copy link
Member Author

roji commented May 5, 2025

We should definitely be stepping up the memes on PRs ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kernel Issues or pull requests impacting the core kernel memory .NET Issue or Pull requests regarding .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants