Skip to content

Refinements - #79

Merged
OnixTech merged 6 commits into
masterfrom
50-refinements
Sep 3, 2026
Merged

OnixTech merged 6 commits into
masterfrom
50-refinements

Conversation

@semere-meb

Copy link
Copy Markdown
Member

Summary

Extensive collection of improvements -- typing, visual improvement, variable names and code organization.

Fixes #

Changes

  • Using Direction enum instead of letters, hence removed all the type-checking ignores
  • Removed redundant Adapter.path_dirs
  • Empty corners(in imperfect mazes) removed
  • Pattern cells recognition improved
  • Removed unnecessary render_text_gen()
  • Several method, variables and parameter names improved

Testing

  • make lint passes locally
  • make test passes locally
  • Manually tested the behavior described above

Checklist

  • PR title follows <TICKET-ID>: short description
  • Linked to the correct issue (Fixes #... above)
  • No unrelated changes bundled into this diff
  • Updated README if this changes user-facing behavior (config keys, controls, etc.)

@semere-meb
semere-meb requested a review from OnixTech September 3, 2026 14:06
@semere-meb semere-meb self-assigned this Sep 3, 2026

@OnixTech OnixTech left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great!

@OnixTech
OnixTech merged commit 57bc663 into master Sep 3, 2026
2 checks passed
@semere-meb
semere-meb deleted the 50-refinements branch September 3, 2026 15:45
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