Update range-loop.odin - #1
Conversation
Odin's index value comes second, not first (as you would see in Go) (see https://odin-lang.org/docs/overview/#range-based-for-loop)
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe range-loop example now binds its iteration components to ChangesRange loop example
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Thanks for putting together the odinbyexample site -- I found it earlier today, and it's nice to see the start to this. Just a minor note here:
Odin's index value comes second, not first (as you would see in Go). See https://odin-lang.org/docs/overview/#range-based-for-loop for the examples there, under the "Alternatively a second index value can be added" section.