Skip to content
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

Fix typ comment in ConnectionPoolConfiguration #202

Merged
merged 1 commit into from
Nov 23, 2023
Merged

Conversation

kommadot
Copy link
Contributor

@kommadot kommadot commented Nov 23, 2023

I found a typo in the variable name in the comment and corrected it.

Make sure that:

  • You have read the contribution guidelines.
  • You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request. (it is very trivial change)
  • You use the code formatters provided here and have them applied to your changes. Don't submit any formatting related changes.
  • You submit test cases (unit or integration tests) that back your changes.

Issue description

New Public APIs

Additional context

The variable name in the comment is maxsize, so we modified it to initialsize.
@mp911de mp911de changed the title Modify comment in ConnectionPoolConfiguration.java Fix typ comment in ConnectionPoolConfiguration Nov 23, 2023
@mp911de mp911de added the type: documentation A documentation update label Nov 23, 2023
@mp911de mp911de added this to the 1.0.2.RELEASE milestone Nov 23, 2023
@mp911de mp911de merged commit 3ee034a into r2dbc:main Nov 23, 2023
1 check failed
@mp911de
Copy link
Member

mp911de commented Nov 23, 2023

Thank you for your contribution. That's merged now.

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

Successfully merging this pull request may close these issues.

None yet

2 participants