Back to all reviewers

Document code purpose

semgrep/semgrep
Based on 4 comments
Python

Ensure that code elements have clear documentation explaining their purpose, behavior, and context. This includes function parameters (especially non-obvious ones), complex business logic, and magic values or constants.

Documentation Python

Reviewer Prompt

Ensure that code elements have clear documentation explaining their purpose, behavior, and context. This includes function parameters (especially non-obvious ones), complex business logic, and magic values or constants.

Key areas to document:

  • Function parameters: Add docstrings or comments explaining what parameters do, when they should be used, and their expected behavior
  • Complex logic: Add explanatory comments before or within complex code sections that describe what the code does and why it’s necessary
  • Magic values: Document the source, purpose, and meaning of hardcoded strings, numbers, or constants

Example from the discussions:

def add_engine_config(
    self, engineType: "EngineType", secrets_origins: Optional[str]
) -> None:
    """
    Configure engine with type and secrets information.
    
    Args:
        engineType: The type of engine to configure
        secrets_origins: Source information for secrets detection, 
                        e.g., "github" or "local_scan"
    """

This practice helps future developers (including yourself) understand the intent behind code decisions and reduces the cognitive load when maintaining or extending the codebase.

4
Comments Analyzed
Python
Primary Language
Documentation
Category

Source Discussions