Skip to content

Clean up codebase with regards to deprecated API #360

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

Merged
merged 1 commit into from
May 26, 2023
Merged

Conversation

tortmayr
Copy link
Contributor

Ensure that deprecated Action & SModel API is encapsulated and do not leak into non-deprecated API by

  • Migrating the examples to the new Action/SModel API
  • Migrating the test cases to the new Action/SModel API
  • Remove any remaining usage of deprecated Action/SModel API in the non-deprecated code base

In addition, this change restructures all files that contain deprecated Action/SModel API and moves the deprecated definitions in a separate section on the end of the file.

This enables easy removal of the deprecated API with the 1.0.0 release. The deprecation section at the file end can be safely removed without breaking codebase.

@tortmayr tortmayr requested review from dhuebner and spoenemann May 25, 2023 21:01
Copy link
Contributor

@spoenemann spoenemann left a comment

Choose a reason for hiding this comment

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

Very helpful cleanup, thank you!

Ensure that deprecated Action & SModel API is encapsulated and do not leak into non-deprecated API by
- Migrating the examples to the new Action/SModel API
- Migrating the test cases to the new Action/SModel API
- Remove any remaining usage of deprecated Action/SModel API in the non-deprecated code base

In addition, this change restructures all files that contain deprecated Action/SModel API and moves the
deprecated definitions in a separate section on the end of the file.

This enables easy removal of the deprecated API with the 1.0.0 release.
The deprecation section at the file end can be safely removed without breaking codebase.
@tortmayr tortmayr merged commit c119b35 into master May 26, 2023
@tortmayr tortmayr deleted the clean-up branch May 26, 2023 08:39
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