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

Update d.json #9011

Closed
wants to merge 1 commit into from
Closed

Update d.json #9011

wants to merge 1 commit into from

Conversation

JohhannasReyn
Copy link
Contributor

Added DuplExpellio to the plugin repository

  • I'm the package's author and/or maintainer.
  • I have read the docs.
  • I have tagged a release with a server version number.
  • My package repo has a description and a README describing what it's for and how to use it.
  • My package doesn't add context menu entries. *
  • My package doesn't add key bindings. **
  • Any commands are available via the command palette.
  • Preferences and keybindings (if any) are listed in the menu and the command palette, and open in split view.

My package is ...

There are no packages like it in Package Control. <--- This one

Added DuplExpellio to the plugin repository
@braver
Copy link
Collaborator

braver commented Nov 30, 2024

Thanks for your submission. Some remarks:

  • If I understand correctly, the behavior is toggled via a setting. Wouldn't it be easier if there were two commands: one to select all duplicates, one to delete them?
  • I also don't fully understand the use case yet. What is considered a duplicate? A character, a word, a line? The threshold setting is also a bit mystifying to me.

Copy link
Collaborator

@packagecontrol-bot packagecontrol-bot left a comment

Choose a reason for hiding this comment

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

Automated testing result: SUCCESS

Repo link: DuplExpellio

Packages added:
  - DuplExpellio

Processing package "DuplExpellio"
  - All checks passed

@braver braver closed this Nov 30, 2024
@JohhannasReyn JohhannasReyn deleted the patch-1 branch January 10, 2025 23:25
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.

3 participants