fix: filter null identifiers

[admy7]

Summary by Sourcery

Fix the issue of null identifiers being included in the list of dataset IDs by adding a filter to remove null values.

Bug Fixes:

• Filter out null identifiers from the list of dataset IDs collected from CKAN packages.

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request addresses a potential issue with null identifiers in the dataset collection process. The change adds a filter to remove null identifiers from the stream of CkanPackage objects before converting them to a list.

File-Level Changes

Change	Details	Files
Added null check filter in dataset identifier collection	Inserted a filter step in the stream pipeline to remove null identifiers Used Objects::nonNull as the predicate for the filter operation	src/main/java/io/github/genomicdatainfrastructure/discovery/datasets/infrastructure/ckan/persistence/CkanDatasetIdsCollector.java

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• Continue your discussion with Sourcery by replying directly to review comments.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[sourcery-ai]

Hey @admy7 - I've reviewed your changes and they look great!

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}