Skip to content

[DOCS] Fix parameter name mismatches in docstrings#11944

Open
eeshsaxena wants to merge 1 commit into
fivetran:developfrom
eeshsaxena:fix-docstring-param-names
Open

[DOCS] Fix parameter name mismatches in docstrings#11944
eeshsaxena wants to merge 1 commit into
fivetran:developfrom
eeshsaxena:fix-docstring-param-names

Conversation

@eeshsaxena

Copy link
Copy Markdown

Fixes three docstrings where the documented parameter name does not match the actual function signature:

File Function Docstring said Signature has
great_expectations/render/util.py substitute_none_for_missing kwargs_list kwarg_list
great_expectations/experimental/metric_repository/batch_inspector.py compute_metric_list_run metrics_list metric_list
great_expectations/experimental/metric_repository/metric_list_metric_retriever.py _calculate_table_metrics metrics_list metric_list

The sibling methods in metric_list_metric_retriever.py that genuinely take metrics_list are left unchanged.

Doc-only change; no behavior modified.

  • Description of PR changes above includes a link to an existing GitHub issue (n/a - trivial doc fix)
  • PR title is prefixed with one of: [BUGFIX], [FEATURE], [DOCS], [MAINTENANCE], [CONTRIB]
  • Code is linted (doc-only change)
  • Appropriate tests and docs have been updated (n/a - docstring-only)

- render/util.py: `kwargs_list` -> `kwarg_list` in
  substitute_none_for_missing
- experimental/metric_repository/batch_inspector.py: `metrics_list` ->
  `metric_list` in compute_metric_list_run
- experimental/metric_repository/metric_list_metric_retriever.py:
  `metrics_list` -> `metric_list` in _calculate_table_metrics
@netlify

netlify Bot commented Jul 3, 2026

Copy link
Copy Markdown

👷 Deploy request for niobium-lead-7998 pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 26eb752

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.

1 participant