Skip to content

Docstrings - #2

Merged
davebelais merged 2 commits into
mainfrom
Update-docstrings
Sep 1, 2025
Merged

Docstrings#2
davebelais merged 2 commits into
mainfrom
Update-docstrings

Conversation

@davebelais

Copy link
Copy Markdown
Contributor

No description provided.

@davebelais
davebelais requested a review from Copilot September 1, 2025 23:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes unused API key authentication functionality and simplifies operation descriptions in the provider data client. The changes focus on cleaning up redundant documentation and removing obsolete authentication parameters.

  • Removes api_key parameter from client initialization and related documentation
  • Simplifies docstrings for datastore query operations by removing verbose POST endpoint references
  • Updates OpenAPI specification to remove redundant operation descriptions

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
tests/conftest.py Removes unused os import and api_key parameter from test client initialization
src/cmsgov/provider_data/v1/client.py Removes api_key parameter from constructor and simplifies method docstrings
scripts/remodel.py Adds logic to clean up operation descriptions and comments out api_key parameter
openapi/provider_data/v1/fixed.json Removes verbose descriptions from datastore query operations

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread tests/conftest.py
Comment thread scripts/remodel.py
@davebelais
davebelais merged commit ee4e363 into main Sep 1, 2025
16 checks passed
@davebelais
davebelais deleted the Update-docstrings branch September 1, 2025 23:07
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