zowe / zowe/zowe-client-python-sdk

[core] Investigate if `warnings.warn` calls can be replaced with INFO level logging

Open
#270 2 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

bug priority-low severity-low
Dominant language
Python
Stars
44
Forks
38
Avg merge
1d 3h
Merged PRs (30d)
1

Description

Now that we have logging in place, we might be able to remove the suppress_config_file_warnings flag and log some of the config file warnings at INFO level instead.

However I think that removing warnings that were already present would be outside the scope of the effort to add a logger, so this could be addressed in a separate issue 🙂

Originally posted by @t1m0thyj in https://github.com/zowe/zowe-client-python-sdk/pull/268#discussion_r1618704407

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.

Research direction

No file or test is named. Start by locating the warnings.warn calls related to config-file warnings, the suppress_config_file_warnings flag, and the existing logger. Review their current behavior and document or implement which warnings can safely use INFO-level logging, with the flag's future status made explicit.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.