NCAS-CMS / NCAS-CMS/cf-plot

Replace `Warning` usage with appropriate specific warnings

Open
#22 0 comments 0 reactions 1 assignee View on GitHub

@sadielbartholomew is already working on this.

Since Feb 23, 2024.

code hygiene
Dominant language
Python
Stars
21
Forks
11
Avg merge
7h 11m
Merged PRs (30d)
1

Description

There are ~20 cases where a Warning is raised in the code:

$ pwd
/home/slb93/git-repos/cf-plot/cfplot
$ git grep "raise Warning" | wc -l
21

These should be made more specific to categorise them, notably in most cases it seems from a quick look that they should each be upgraded to a UserWarning to make it clear it isn't something dodgy about the environment or code, but rather a message to the user about the context of cf-plot and the underlying matplotlib usage, etc.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.