Skip to content

fix : move unreachable docstring in RegistryValidator.report and fix --strict flag to treat warnings as errors #1857

Description

@tmdeveloper007

Summary of What Needs to be Done

In , the method has two bugs:

  1. The docstring is placed AFTER the statement, making it an unreachable statement instead of a proper docstring.
  2. The flag (at the bottom of the file) only checks to set the exit code, but should treat warnings as errors too. Currently is dead code.

Changes that Need to be Made

  1. Move the docstring to the beginning of the method (before the block).
  2. Fix the logic to exit with code 1 when warnings exist in strict mode.

Impact that it would Provide

  • Proper docstring helps developers understand the method.
  • Correct behavior ensures warnings are properly escalated in CI/CD pipelines.

Note: Please assign this issue to the tmdeveloper007 account.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions