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

Dataflow: Delete the old configuration-class based api. #18191

Merged
merged 17 commits into from
Dec 4, 2024

Conversation

aschackmull
Copy link
Contributor

🎉 🎉 This deletes the old and long-deprecated data flow api that was based on extending a configuration class.

See also the blog post https://github.blog/changelog/2023-08-14-new-dataflow-api-for-writing-custom-codeql-queries/ where we announced that this would happen now.

hvitved
hvitved previously approved these changes Dec 3, 2024
Copy link
Contributor

@hvitved hvitved left a comment

Choose a reason for hiding this comment

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

Ruby / C# 👍

hvitved
hvitved previously approved these changes Dec 3, 2024
owen-mc
owen-mc previously approved these changes Dec 3, 2024
Copy link
Contributor

@owen-mc owen-mc left a comment

Choose a reason for hiding this comment

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

Go 👍🏻

@aschackmull aschackmull dismissed stale reviews from owen-mc and hvitved via 5b51088 December 3, 2024 14:54
@aschackmull aschackmull force-pushed the dataflow/remove-deprecated-lib branch 2 times, most recently from 92c1c44 to fce737f Compare December 3, 2024 15:25
---
category: breaking
---
* Deleted the old deprecated data flow api that was based on extending a configuration class.
Copy link
Collaborator

@adityasharad adityasharad Dec 3, 2024

Choose a reason for hiding this comment

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

Something like this will help any remaining users migrate.

Suggested change
* Deleted the old deprecated data flow api that was based on extending a configuration class.
* Deleted the old deprecated data flow API that was based on extending a configuration class. See https://github.blog/changelog/2023-08-14-new-dataflow-api-for-writing-custom-codeql-queries for instructions on migrating your queries to use the new API.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@aschackmull aschackmull force-pushed the dataflow/remove-deprecated-lib branch 2 times, most recently from 91f4c6a to aee18c7 Compare December 3, 2024 18:56
@aschackmull aschackmull force-pushed the dataflow/remove-deprecated-lib branch from aee18c7 to 8c99ad4 Compare December 3, 2024 19:09
Copy link
Contributor

@jketema jketema left a comment

Choose a reason for hiding this comment

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

C++/Swift 👍

Copy link
Contributor

@yoff yoff left a comment

Choose a reason for hiding this comment

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

Python 👍 we are just deleting several deprecated classes :-)

Copy link
Contributor

@owen-mc owen-mc left a comment

Choose a reason for hiding this comment

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

Go (again, as there were more changes) 👍🏻

@aschackmull
Copy link
Contributor Author

Alrighty, let's merge this!

@aschackmull aschackmull merged commit 03fdceb into github:main Dec 4, 2024
58 checks passed
@aschackmull aschackmull deleted the dataflow/remove-deprecated-lib branch December 4, 2024 10:31
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.

6 participants