Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: wezterm cwd compatibility #439

Merged
merged 2 commits into from
Oct 9, 2023
Merged

feat: wezterm cwd compatibility #439

merged 2 commits into from
Oct 9, 2023

Conversation

kennycallado
Copy link
Contributor

This PR makes joshuto compatible with wezterm at least with the way it manages the current directory updates.

  1. Added a new cwd utility module.
  2. Centralized CWD updates using the cwd::set_current_dir function.
  3. Updated to use the cwd utility for CWD management.
  4. Implements OSC 7: Escape sequences to inform the terminal about the current directory ref.

@kamiyaa
Copy link
Owner

kamiyaa commented Oct 9, 2023

Looks good

@kamiyaa kamiyaa merged commit 7aec4b6 into kamiyaa:main Oct 9, 2023
Dzordzu pushed a commit to Dzordzu/joshuto that referenced this pull request Oct 14, 2023
@kennycallado
feat: wezterm cwd compatibility (kamiyaa#439)
2fbc5b2 
* add: new cwd utility

* Use cwd::set_current_dir to update terminal with current working directory

---------

Co-authored-by: kennycallado <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kamiyaa kamiyaa kamiyaa approved these 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
@kennycallado @kamiyaa