eclipse-collections icon indicating copy to clipboard operation
eclipse-collections copied to clipboard

Improved Documentation for the Predicate Interface

Open BrijeshPatra opened this issue 2 years ago • 3 comments

Description:

  • The current documentation for the Predicate interface in Eclipse Collections contained inaccuracies and could benefit from improvements. This pull request aims to enhance the clarity and accuracy of the Predicate interface documentation.

Changes Made:

  • Revised the description of the Predicate interface to provide a more accurate and informative definition.
  • Removed the specific mention of lambdas or closures to avoid potential confusion.
  • Ensured consistency in terminology and formatting.
  • Added a brief explanation of the purpose of Predicate and its usage.

Context:

  • The Predicate interface is a fundamental component of Eclipse Collections, and accurate documentation is essential for users to understand its role and use it effectively.

Testing:

  • I have reviewed and tested the changes to ensure they accurately reflect the intended improvements in the documentation.

Screenshots:

Screenshot (493)

Request for Review:

  • I kindly request a review of these documentation improvements. Your feedback and suggestions for further enhancements are highly appreciated. @sullis @brunoborges

  • Thank you for your attention to this contribution!

BrijeshPatra avatar Aug 24 '23 15:08 BrijeshPatra

Thank you very much. Here are my remarks:

  1. if "filtering and "discriminating" are used as synonyms, I would recommend to substitute "for filtering or discriminating" with either:
    • "for filtering (discriminating)"
    • "for filtering/discriminating"
    • "for filtering (also called discriminating)"
  2. This changes multiples files in .idea/, which I suppose is unintentional.
  3. Predicate can treat objects, but it can treat any type, including primitives. This problem is basically pre-existing, but it would be better to talk about "elements" than objects.
  4. The last sentence should end with a full stop (".").

By the way, I don't think including a screenshot is necessary in this case.

Chealer avatar Sep 05 '23 12:09 Chealer

Thank you for your remarks @Chealer I will surely try to improvise again in the documentation part.

BrijeshPatra avatar Sep 06 '23 11:09 BrijeshPatra

Great It is my pleasure @BrijeshPatra

Chealer avatar Sep 06 '23 21:09 Chealer