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 punctuation and minor Javadoc issues #409

Merged
merged 1 commit into from
Jul 29, 2023

Conversation

mawiesne
Copy link
Contributor

@mawiesne mawiesne commented Jul 29, 2023

  • fixes misplaced symbols in JavaDoc (mainly in Hasher, CountingLongPredicate)
  • resolves grammar and punctuation issues

@mawiesne
Copy link
Contributor Author

Ping @garydgregory @kinow

@codecov-commenter
Copy link

codecov-commenter commented Jul 29, 2023

Codecov Report

Merging #409 (0668de8) into master (8a0d6b0) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #409   +/-   ##
=========================================
  Coverage     81.20%   81.20%           
  Complexity     4625     4625           
=========================================
  Files           290      290           
  Lines         13483    13483           
  Branches       1993     1993           
=========================================
  Hits          10949    10949           
  Misses         1933     1933           
  Partials        601      601           
Files Changed Coverage Δ
...java/org/apache/commons/collections4/MapUtils.java 93.30% <ø> (ø)
...rg/apache/commons/collections4/PredicateUtils.java 83.87% <ø> (ø)
...org/apache/commons/collections4/SplitMapUtils.java 73.58% <ø> (ø)
.../apache/commons/collections4/TransformerUtils.java 100.00% <ø> (ø)
.../collections4/collection/PredicatedCollection.java 88.13% <ø> (ø)
...he/commons/collections4/functors/WhileClosure.java 80.00% <ø> (ø)
.../commons/collections4/iterators/IteratorChain.java 86.27% <ø> (ø)
...mmons/collections4/iterators/NodeListIterator.java 83.33% <ø> (ø)
...ns/collections4/iterators/ObjectGraphIterator.java 100.00% <ø> (ø)
...ommons/collections4/list/CursorableLinkedList.java 83.33% <ø> (ø)
... and 7 more

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

Copy link
Member

@garydgregory garydgregory left a comment

Choose a reason for hiding this comment

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

@mawiesne
Thank you for your PR.
I am worried that this will clash with ongoing work in PRs in the bloom filter package.
Would you please update the PR to not touch that package?

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 @mawiesne !

…dicate)

resolves grammar and punctuation issues
@mawiesne mawiesne force-pushed the fix-grammar-and-javadoc-issues branch from d82c18e to 0668de8 Compare July 29, 2023 19:50
@mawiesne
Copy link
Contributor Author

Would you please update the PR to not touch that package?

I did - changes in bloomfilter package reverted.

@garydgregory garydgregory merged commit 33bd5d1 into apache:master Jul 29, 2023
6 checks passed
@garydgregory garydgregory changed the title Fix punctuation and minor JavaDoc issues Fix punctuation and minor Javadoc issues Jul 29, 2023
garydgregory added a commit that referenced this pull request Jul 29, 2023
@mawiesne mawiesne deleted the fix-grammar-and-javadoc-issues branch July 30, 2023 04:53
anantdamle pushed a commit to anantdamle/commons-collections that referenced this pull request Aug 14, 2023
anantdamle pushed a commit to anantdamle/commons-collections that referenced this pull request Aug 14, 2023
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