Skip to content

Conversation

@mawiesne
Copy link
Contributor

@mawiesne mawiesne commented Aug 26, 2023

As the title says.

@mawiesne
Copy link
Contributor Author

@kinow @garydgregory FYI.

@codecov-commenter
Copy link

Codecov Report

Merging #299 (d0a5d1e) into master (95ff23f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #299   +/-   ##
=========================================
  Coverage     59.77%   59.77%           
  Complexity     1785     1785           
=========================================
  Files            57       57           
  Lines          7415     7415           
  Branches        419      419           
=========================================
  Hits           4432     4432           
  Misses         2767     2767           
  Partials        216      216           
Files Changed Coverage Δ
...org/apache/commons/dbcp2/DelegatingConnection.java 43.79% <ø> (ø)
.../org/apache/commons/dbcp2/DelegatingResultSet.java 45.61% <ø> (ø)
...a/org/apache/commons/dbcp2/PoolableConnection.java 68.46% <ø> (ø)
...pache/commons/dbcp2/PoolablePreparedStatement.java 85.18% <ø> (ø)
...va/org/apache/commons/dbcp2/PoolingConnection.java 81.70% <ø> (ø)
...ache/commons/dbcp2/cpdsadapter/ConnectionImpl.java 50.76% <ø> (ø)
...ommons/dbcp2/cpdsadapter/PooledConnectionImpl.java 52.07% <ø> (ø)
.../dbcp2/datasources/KeyedCPDSConnectionFactory.java 65.85% <ø> (ø)
...mmons/dbcp2/datasources/PerUserPoolDataSource.java 88.95% <ø> (ø)
...ommons/dbcp2/managed/LocalXAConnectionFactory.java 90.24% <ø> (ø)
... and 2 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@kinow kinow left a comment

Choose a reason for hiding this comment

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

Thank you!

@garydgregory garydgregory merged commit dc2cb63 into apache:master Aug 26, 2023
@mawiesne mawiesne deleted the fix-typos-and-grammar-issues branch August 27, 2023 05:00
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.

4 participants