-
Notifications
You must be signed in to change notification settings - Fork 44
Remove support for ~/.esmvaltool/config-user.yml and ~/.esmvaltool/dask.yml
#2878
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
Conversation
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (92.30%) is below the target coverage (100.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #2878 +/- ##
==========================================
- Coverage 95.49% 95.44% -0.05%
==========================================
Files 261 261
Lines 15557 15415 -142
==========================================
- Hits 14856 14713 -143
- Misses 701 702 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
valeriupredoi
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
excellent! But I reckon we need a troubleshooting example - I asked for one in #2765 - so maybe pop one in there 🍺
|
Thanks for reviewing! Can you explain in a bit more detail what you expect from "a troubleshooting example"? |
ah sorry, stuff like unsetting ESMVALTOOL_CONFIG_DIR variable, and removing an existing |
|
After this is merged,
|
that would be most welcome, good sir 🍺
yes indeed! I never thought of it that way, mostly because I barely run the tool at all these days 😁 |
Description
Remove support for
~/.esmvaltool/config-user.ymland~/.esmvaltool/dask.yml. These options have been deprecated since v2.12.0 of the tool and will no longer be available starting in v2.14.0.Related to #2371
Link to documentation: https://esmvaltool--2878.org.readthedocs.build/projects/ESMValCore/en/2878/quickstart/configure.html#yaml-files (note the deleted box describing the ~/.esmvaltool/config-user.yml feature).
Upgrade instructions
Move the file
~/.esmvaltool/config-user.ymlto~/.config/esmvaltool/config-user.ymland replace~/.esmvaltool/dask.ymlby new way of configuring Dask.Before you get started
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.
Changes are backward compatibleTo help with the number pull requests: