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 Alt + E open currt folder And Fast Print Variable Plug #8932

Closed
wants to merge 1 commit into from

Conversation

mucpsing
Copy link

@mucpsing mucpsing commented Jun 5, 2024

  • I'm the package's author and/or maintainer.
  • I have have read [the docs][1].
  • I have tagged a release with a [semver][2] 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.
  • If my package is a syntax it doesn't also add a color scheme. ***
  • If my package is a syntax it is named after the language it supports (without suffixes like "syntax" or "highlighting").
  • I use [.gitattributes][3] to exclude files from the package: images, test files, sublime-project/workspace.

This is a basic plugin with the following main features:

When using the shortcut key alt+e, it will invoke the file manager to quickly open the directory where the current file is located.
A right-click menu item Cps Print Currt Param is added, which allows for quickly printing the variable selected by the current cursor (currently only supports Python and JavaScript).
It integrates some commonly used shell invocation methods (under Windows) for quick testing and debugging.

@braver
Copy link
Collaborator

braver commented Jun 26, 2024

Hi, thanks for your submission. It looks like your package is a bit like a compilation of features that are useful in a given context? There are various packages that seem to provide similar features separately already and we don't usually approve packages compilations. As a whole, it's also a bit much to review, for the features themselves and even just basics around key bindings, context menu entries and the entry here (check the build output).

@braver braver added the stale The pull request needs to be updated but has not been within the recent past (2 weeks) label Aug 7, 2024
@braver
Copy link
Collaborator

braver commented Aug 7, 2024

Please respond to continue the review.

@braver braver closed this Aug 21, 2024
@braver braver added the timeout A pull request needed changes but was not updated in time (2 weeks after becoming stale) label Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feedback provided stale The pull request needs to be updated but has not been within the recent past (2 weeks) timeout A pull request needed changes but was not updated in time (2 weeks after becoming stale)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants