Skip to content

refactor(list): make foldi iterative#3351

Merged
bobzhang merged 1 commit intomainfrom
refactor/list-foldi-iterative
Mar 26, 2026
Merged

refactor(list): make foldi iterative#3351
bobzhang merged 1 commit intomainfrom
refactor/list-foldi-iterative

Conversation

@bobzhang
Copy link
Copy Markdown
Contributor

@bobzhang bobzhang commented Mar 26, 2026

Summary

  • Replace the tail-recursive inner go function in foldi with a for-loop
  • Simplifies the code and avoids unnecessary stack growth
  • Pure refactor with no interface changes

Validation

  • moon check list
  • moon test list (195 tests pass)

🤖 Generated with Claude Code


Open with Devin

Replace the tail-recursive inner `go` function in foldi with a
for-loop, simplifying the code and avoiding unnecessary stack growth.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@coveralls
Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 3180

Details

  • 3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.553%

Totals Coverage Status
Change from base Build 3174: 0.0%
Covered Lines: 14073
Relevant Lines: 14728

💛 - Coveralls

@bobzhang bobzhang merged commit 2374377 into main Mar 26, 2026
14 checks passed
@bobzhang bobzhang deleted the refactor/list-foldi-iterative branch March 26, 2026 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants