Skip to content

Remove deprecated features esmvaltool config get_config_user, esmvaltool config get_config_developer, esmvalcore.local.DataSource - #3210

Merged
schlunma merged 4 commits into
mainfrom
remove_deprecated_features
Aug 27, 2026
Merged

Remove deprecated features esmvaltool config get_config_user, esmvaltool config get_config_developer, esmvalcore.local.DataSource#3210
schlunma merged 4 commits into
mainfrom
remove_deprecated_features

Conversation

@schlunma

@schlunma schlunma commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Description

This PRs removes some of the deprecated features that are scheduled for removal in v2.16.0. There are a lot of them, so I thought its best to do this piece by piece.

Backwards-incompatible change #

This PR removes deprecated features that are scheduled for removal in v2.16.0.

esmvaltool config get_config_user

Deprecated in #2765. Use esmvaltool config copy instead.

esmvaltool config get_config_developer

The config-developer.yml file has been deprecated in #2946. The corresponding option are now configured in ESMValCore's configuration. These configuration files can be copied via esmvaltool config copy instead.

esmvalcore.local.DataSource

Deprecated in #2911. Use esmvalcore.io.local.LocalDataSource instead.


Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.


To help with the number pull requests:

@codecov

codecov Bot commented Aug 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.45%. Comparing base (936450c) to head (870519f).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3210      +/-   ##
==========================================
- Coverage   96.50%   96.45%   -0.06%     
==========================================
  Files         281      281              
  Lines       16360    16317      -43     
==========================================
- Hits        15789    15739      -50     
- Misses        571      578       +7     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bouweandela bouweandela left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, though esmvalcore.local.DataSource seems a bit unrelated to the command line options that are removed in this pull request, so it may look nicer in the changelog to make it two separate PRs.

@schlunma

Copy link
Copy Markdown
Contributor Author

Yeah, agreed. Originally, I wanted to handle all cases in a single PR, but quickly realized that there are way too many. If you prefer, I can open a separate PR, but we could always manually adapt the changelog (we do this for the backwards-incompatible changes anyway).

@schlunma

Copy link
Copy Markdown
Contributor Author

Merging this now to allow review of #3211.

@schlunma
schlunma merged commit 8c8ce25 into main Aug 27, 2026
6 checks passed
@schlunma
schlunma deleted the remove_deprecated_features branch August 27, 2026 12:43
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.

2 participants