creativej/sublimeshellexec

A simple plugin for hooking up shell commands in Sublime.

sublimeshellexec's Issues

Issues to address from package control PR

• Don't hardcode folder separators
• Don't override default types, not even locally. This also happens a lot with file btw.
• Use with open(file, "w") as f: is one of the best things in Python
• There could be a better way to get the current function name in python 3?
• Use __package__ instead of very long construct using the inspect module. Can just access __file__ for most cases. (There is also __package__ which will be more useful for the next point.)
• Plugin won't not work currently when installed by Package Control. This is because all packages are zipped by default and stored in the Installed Packages directory.
  Possible solution:

you can override specific files from within the Packages dir and a folder with the same name as the corresponding PackageName.sublime-package - without extension ofc. So, what you should do is to check whether os.path.join(sublime.packages_path(), __package__) exists, create that if necessary and then write into a file there.

More details wbond/package_control_channel#3391

Issues to address from package control PR

• Don't hardcode folder separators
• Don't override default types, not even locally. This also happens a lot with file btw.
  with open(file, "w") as f: is one of the best things in Python
• There could be a better way to get current function name in Python 3?
• Use package instead of very long construct using the inspect module. Can just access file for most cases. (There is also package which will be more useful for the next point.)
• Plugin won't not work currently when installed by Package Control. This is because all packages are zipped by default and stored in the Installed Packages directory.
  Possible solution:

you can override specific files from within the Packages dir and a folder with the same name as the corresponding PackageName.sublime-package - without extension ofc. So, what you should do is to check whether os.path.join(sublime.packages_path(), package) exists, create that if necessary and then write into a file there.