Fixed LazyList for Oai Files.

[mdellabitta]

Important

Refactor localHarvest() in OaiFileHarvester.scala to use foreach instead of for loop for LazyList processing.

• Refactor:
  • Change for loop to foreach in localHarvest() method of OaiFileHarvester.scala for processing LazyList elements.

^{This description was created by for f992060. It will automatically update as commits are pushed.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to f992060 in 14 seconds

More details

• Looked at 20 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. src/main/scala/dpla/ingestion3/harvesters/file/OaiFileHarvester.scala:154

• Draft comment:
  Consider using a more descriptive variable name instead of result for clarity, such as fileResult. This applies to the foreach loop on line 154.
• Reason this comment was not posted:
  Confidence changes required: 30%
  The change from a for-loop to a foreach is appropriate for side-effect operations, but the code can be further improved for readability and consistency.

Workflow ID: wflow_lsvJMqgroDtDn6Xn

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.