Skip to content

chore: code logic separation and clean up#114

Merged
playdohface merged 5 commits intozed-extensions:mainfrom
tartarughina:clean-up
Nov 2, 2025
Merged

chore: code logic separation and clean up#114
playdohface merged 5 commits intozed-extensions:mainfrom
tartarughina:clean-up

Conversation

@tartarughina
Copy link
Copy Markdown
Collaborator

@tartarughina tartarughina commented Nov 1, 2025

Improving the repo code readability and maintainability by:

  • Dividing the code into logic blocks.
    • Before, the bulk of the code was under java.rs. Now that code has been aggregated into specific files
  • Deduplicate code into reusable functions
    • General code that can be used by multiple files has been grouped into a single util.rs file

Feature:

  • Add ~ expansion for Unix-like systems under util.rs

- Divide the code into logic blocks
- Deduplicate code into reusable functions
@cla-bot cla-bot Bot added the cla-signed label Nov 1, 2025
Copy link
Copy Markdown
Collaborator

@playdohface playdohface left a comment

Choose a reason for hiding this comment

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

Nice, this works for me. At this point I'm wondering if we even need the chatty comments anymore, but they are somewhat charming. But clippy has a bunch of helpful things to say still, we might want to incorporate that

@playdohface playdohface merged commit 1991fc1 into zed-extensions:main Nov 2, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants