jdaviz icon indicating copy to clipboard operation
jdaviz copied to clipboard

TST: Remove blanket ignore for test warnings and fix failures

Open Jdaviz-Triage-Bot opened this issue 2 years ago • 2 comments

Reporter: pllim

https://github.com/spacetelescope/jdaviz/blob/639d22e3fc45c3457e43f70fbcd454e3e05437a2/pyproject.toml#L140-L149

The blanket ignores can accidentally ignore things we care about (e.g., #2460). We should remove these and handle any remaining warnings individually as appropriate:

a. fix the problem, b. ignore it locally only for the affected line of code in tests, or c. a more specific blanket ignore

If this turns out to be too much work, might have to split into smaller tickets focusing only on a subset of these.

🐱


DISCLAIMER: This issue was autocreated by the Jdaviz Issue Creation Bot on behalf of the reporter.

Jdaviz-Triage-Bot avatar Sep 14 '23 18:09 Jdaviz-Triage-Bot

This issue is tracked on JIRA as JDAT-3810.

stscijgbot-jwql avatar Jul 09 '24 19:07 stscijgbot-jwql

Comment by Kyle Conroy on JIRA:

NOTICE: A corresponding GitHub issue was automatically created: #2461

stscijgbot-jwql avatar Jul 09 '24 19:07 stscijgbot-jwql