Skip to content

All: delete outdated deprecations#18601

Merged
erik-krogh merged 9 commits into
github:mainfrom
erik-krogh:del-deps-jan-2025
Jan 28, 2025
Merged

All: delete outdated deprecations#18601
erik-krogh merged 9 commits into
github:mainfrom
erik-krogh:del-deps-jan-2025

Conversation

@erik-krogh

@erik-krogh erik-krogh commented Jan 27, 2025

Copy link
Copy Markdown
Contributor

Simialar to last time: #17349

I recommend the C team look at abe18ad.
That was beyond what I feel comfortable deleting (one thing lead to another...).

There was a bunch of JS deprecations, but I reverted all of them.
Asger committed a lot of deprecations a long time ago, but those only recently made it to main.

…previous naming conflict has been deleted
@erik-krogh erik-krogh marked this pull request as ready for review January 28, 2025 09:28
Copilot AI review requested due to automatic review settings January 28, 2025 09:28
@erik-krogh erik-krogh requested review from a team as code owners January 28, 2025 09:28
@erik-krogh erik-krogh requested a review from a team as a code owner January 28, 2025 09:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot reviewed 63 out of 63 changed files in this pull request and generated no comments.

Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab. Learn more

@jketema

jketema commented Jan 28, 2025

Copy link
Copy Markdown
Contributor

The old C++ dataflow library can simply not be removed at this point, as there are external repos depending on it.

@jketema jketema left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

C++ and Swift 👍

@hvitved hvitved left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

C# and Ruby 👍

@owen-mc owen-mc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Go 👍🏻

@tausbn tausbn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Python 👍

@aschackmull aschackmull left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Java 👍

@aschackmull aschackmull left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Shared code 👍

@erik-krogh erik-krogh merged commit f0755bf into github:main Jan 28, 2025
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.

7 participants