Skip to content

Refactor tree implementation#171

Merged
pointlander merged 5 commits intopointlander:mainfrom
mrclmr:refactor-tree
Apr 15, 2025
Merged

Refactor tree implementation#171
pointlander merged 5 commits intopointlander:mainfrom
mrclmr:refactor-tree

Conversation

@mrclmr
Copy link
Copy Markdown
Contributor

@mrclmr mrclmr commented Apr 14, 2025 •
edited
Loading

  • extract functions to methods
  • isolate recursive functions
  • use separate declaration statements in new lines for variables

pointlander
pointlander requested changes Apr 15, 2025
View reviewed changes
Copy link
Copy Markdown
Owner

@pointlander pointlander left a comment

Choose a reason for hiding this comment

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

Some changes need to be made.

Comment thread tree/peg.go Outdated
Comment thread tree/peg.go Outdated
Comment thread tree/peg.go Outdated
@pointlander pointlander merged commit 766e348 into pointlander:main Apr 15, 2025
1 check passed
@mrclmr mrclmr deleted the refactor-tree branch May 29, 2025 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pointlander pointlander pointlander requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mrclmr @pointlander