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.

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

add clip copy and clip paste #1009

Merged
merged 1 commit into from
Jan 5, 2025
Merged

Conversation

Bahex
Copy link
Contributor

@Bahex Bahex commented Jan 5, 2025

Description

Add clip copy and clip paste for interacting with system clipboard, making use of OSC 52.

I'm not sure how to write tests for these commands.

Tip

No platform specific external binary is required.

Warning

  • Not all terminal emulators will support this
  • Terminal multiplexers may interfere with it, depending on their configuration.

Related

@fdncred fdncred merged commit 66c9995 into nushell:main Jan 5, 2025
1 check passed
@fdncred
Copy link
Collaborator

fdncred commented Jan 5, 2025

Thanks

@@ -0,0 +1,2 @@
export module aggregate
Copy link
Member

Choose a reason for hiding this comment

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

is this line expected? i don't see any mention to this module anywhere 😮

Copy link
Collaborator

@fdncred fdncred Jan 9, 2025

Choose a reason for hiding this comment

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

It's in the std-rfc stdlib-candidate/std-rfc/aggregate/mod.nu, but I'm not sure it's used in clip/copy/paste.

Copy link
Member

Choose a reason for hiding this comment

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

yeah maybe it's legit overall, just does not look like it belongs here 😉

@NotTheDr01ds
Copy link
Contributor

I'm not sure how to write tests for these commands.

Me neither :-/

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.

4 participants