Skip to content

fixing security and code quality reports#79

Merged
advaitpatel merged 1 commit into
mainfrom
improving-code-quality-and-security
May 15, 2026
Merged

fixing security and code quality reports#79
advaitpatel merged 1 commit into
mainfrom
improving-code-quality-and-security

Conversation

@advaitpatel
Copy link
Copy Markdown
Collaborator

Pull Request

Description

Summary of the changes and the related issue. Include motivation and context.

Closes #

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Code style update (formatting, renaming)
  • Code refactoring (no functional changes)
  • Performance improvement
  • Test update
  • Build / CI configuration
  • Security fix

How Has This Been Tested?

Describe the tests run to verify the changes:

  • Unit tests
  • Manual testing

Test Configuration:

  • Python version:
  • Operating System:
  • DockSec version:

Checklist

  • Code follows the style guidelines of this project
  • Self-review completed
  • Hard-to-understand areas are commented
  • Documentation updated where needed
  • No new warnings or errors introduced
  • Tests added that prove the fix or feature works
  • All existing tests pass
  • Dependent changes have been merged and published
  • Spelling checked

Screenshots (if applicable)

Related Issues / PRs

  • Relates to #
  • Depends on #

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

Comment thread docksec/cli.py
import os
import argparse
from typing import NoReturn, Optional
from typing import Optional
Comment thread docksec/cli.py
import os
import argparse
from typing import NoReturn, Optional
from typing import Optional
Copy link
Copy Markdown
Collaborator Author

@advaitpatel advaitpatel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@advaitpatel advaitpatel merged commit a3177a9 into main May 15, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants