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

Remove deprecation limit on ert.data.loader.load_general_data and load.summary.data #3883

Merged

Conversation

lars-petter-hauge
Copy link
Contributor

@lars-petter-hauge lars-petter-hauge commented Sep 9, 2022

Removed the deprecated_on argument. We failed to remove the deprecated functions (the test is a bit hard on failure) before 3.0 was released. Instead of removing the functions now and releasing 3.1, risking that our users will have to update fast, we remove the hard check on version.

The warning of when the function will be removed has still been produced for 19 months.

Pre review checklist

  • Added appropriate release note label
  • PR title captures the intent of the changes, and is fitting for release notes.
  • Commit history is consistent and clean, in line with the contribution guidelines.

Adding labels helps the maintainers when writing release notes. This is the list of release note labels.

Removed the deprecated_on argument. We failed to remove the deprecated
functions (the test is a bit hard on failure) before 3.0 was released.
Instead of removing the functions now and releasing 3.1, risking that
our users will have to update fast, we remove the hard check on version.

The warning of when the function will be removed has still been produced
for 19 months.
@lars-petter-hauge lars-petter-hauge added the release-notes:misc Automatically categorise as miscellaneous change in release notes label Sep 9, 2022
@lars-petter-hauge lars-petter-hauge merged commit bb518ca into equinor:main Sep 9, 2022
@lars-petter-hauge lars-petter-hauge deleted the bump_depr_version branch September 9, 2022 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-notes:misc Automatically categorise as miscellaneous change in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants