[XxFran10xX]

Review-only copy of #83, which was merged before CodeRabbit had reviewed it (CodeRabbit won't review closed PRs). The base is main before #83 (afd9364) and the head is the merged commit 728599e, so the diff is exactly #83. Findings get fixed in a new PR against main. This PR will be closed unmerged and both branches deleted.

@coderabbitai full review

🤖 Generated with Claude Code

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 63431591-9c83-4393-96e5-52d0286503d6

📥 Commits

Reviewing files that changed from the base of the PR and between afd9364 and 728599e.

📒 Files selected for processing: 4

• README.md
• src/main/java/net/tfminecraft/cooking/crops/CropPlantingRule.java
• src/main/resources/crops.yml
• src/test/java/net/tfminecraft/cooking/crops/CropPlantingTest.java

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Summary

Summary by CodeRabbit

• New Features
  • Crops can now grow beneath configured low decorations, including slabs, stairs and signs, up to five blocks above them.
  • Block tags can be used to configure permitted cover materials.
• Documentation
  • Clarified that double slabs count as full-block roofs.

Crop planting rules now support configurable low-cover materials and height limits. Block materials and block tags can define low cover. The sky check treats qualifying cover as open sky, except double slabs.

Changes: Crop Low-Cover Rules

Layer / File(s)	Summary
Configure low-cover materials src/main/java/net/tfminecraft/cooking/crops/CropPlantingRule.java, src/test/java/net/tfminecraft/cooking/crops/CropPlantingTest.java	The rule reads low-cover materials and a non-negative height limit. It accepts block materials and block tags, and logs invalid entries. Tests cover tag expansion and invalid entries.
Apply low-cover rules to sky checks src/main/java/net/tfminecraft/cooking/crops/CropPlantingRule.java, src/main/resources/crops.yml, src/test/java/net/tfminecraft/cooking/crops/CropPlantingTest.java, README.md	The sky check allows configured low cover within the height limit, but double slabs remain roofs. The configuration permits slab, stair and sign tags up to five blocks above crops. Tests check height limits and slab behaviour. The README now includes low pergolas in its crop-growing description.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Feature

Suggested reviewers: ryanbarlow97

Merge Risk: ⚪ Minimal · up to 72859

No actionable code issue was identified. This review-only PR is intended to close unmerged.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[XxFran10xX]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[XxFran10xX]

CodeRabbit approved with no actionable comments. Closing unmerged as planned; the change shipped in v0.5.4.