Skip to content

refactor(BaseService): improve prepare_request() docstring #193

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

padamstx
Copy link
Member

This commit simply modifies the docstring comments related to the headers and params arguments to the BaseService.prepare_request() function in order to clarify that any entries in either of the two dictionary arguments will simply be ignored if they have a value of None.

@padamstx padamstx self-assigned this Apr 17, 2024
@padamstx padamstx requested a review from dpopp07 April 17, 2024 19:07
Copy link

@dpopp07 dpopp07 left a comment

Choose a reason for hiding this comment

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

Looks good! 👍

Base automatically changed from auth-user-agent to main April 17, 2024 19:40
This commit simply modifies the docstring comments related to the
`headers` and `params` arguments to the `BaseService.prepare_request()`
function in order to clarify that any entries in either of the two dictionary
arguments will simply be ignored if they have a value of None.

Signed-off-by: Phil Adams <[email protected]>
@padamstx padamstx merged commit 1c20738 into main Apr 17, 2024
@padamstx padamstx deleted the issue-1798 branch April 17, 2024 19:50
@ibm-devx-sdk
Copy link

🎉 This PR is included in version 3.20.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants