Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

D417 not raised in a standard Google configuration with missing arguments descriptions #655

Open
Alexander-Serov opened this issue Oct 2, 2023 · 0 comments

Comments

@Alexander-Serov
Copy link

Alexander-Serov commented Oct 2, 2023

This is similar to #459, but my bug seems to be even simpler because it doesn't require the presence of extra sections.

I launch

pydocstyle script.py --convention=google --add-ignore=D100

on

# script.py

from typing import List, Tuple, Dict

def update_table_and_indicators2(
    data: dict,
    date_format: str,
    decision_cols: List[str],
    fsl: str,
    scope_toggle: bool,
    scope_own: float,
    scope_comp: float,
    scope_total_own: float,
    scope_total_comp: float,
    rev_unit_toggle: bool,
    keep_subsegments: bool,
    tidy_numbers: bool,
    v: List[str],
) -> Tuple[List[Dict], List[Dict], str]:
    """Updates the table and indicators displayed on the right part of the dashboard.

    Args:
        fsl: aaa

    Returns:
        columns: aaa
        tab: aaa
        dates_considered: aaa
    """
    pass

And the checks pass with no error messages, although most of the input parameters are clearly missing.

Does anyone know if this is the same as #459 or is it something else?

Further details

  • I am not using any external configuration file and this is obtained on pydocstyle==6.3.0
  • Downgrading to 6.0.0 doesn't help
  • Downgrading to 5.0.0 doesn't help
  • Downgrading to 4.0.1 helps and the errors show correctly, so it seems like there was a bug introduced in 5.0.0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant