-
Notifications
You must be signed in to change notification settings - Fork 152
Add SublimeProjectLauncher plugin #496
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
base: plugin_api_v2
Are you sure you want to change the base?
Add SublimeProjectLauncher plugin #496
Conversation
🚨 gitStream Monthly Automation Limit Reached 🚨 Your organization has exceeded the number of pull requests allowed for automation with gitStream. To continue automating your PR workflows and unlock additional features, please contact LinearB. |
Add SublimeProjectLauncher plugin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The metadata for your plugin here is different than the metadata in your plugin's plugin.json
file.
For info about what should be included in your plugin's file in this repository, see the plugin submission instructions
For info about what should be included in your plugin's plugin.json
file, see the docs on the plugin.json
file
@@ -0,0 +1,14 @@ | |||
{ | |||
"ID": "a7b9c10f-df98-42d7-b86f-6589d62f971d", | |||
"Name": "Sublime Project Launcher", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The name here does not match the name in the filename.
Spaces in the filename are allowed, so renaming the file to Sublime Project Launcher-a7b9c10f-df98-42d7-b86f-6589d62f971d.json
is a valid solution
"ExecuteFileName": "main.py", | ||
"Keyword": "sp", | ||
"Website": "https://github.com/Rahulrgd/SublimeProjectLauncher", | ||
"UrlDownload": "https://github.com/Rahulrgd/SublimeProjectLauncher/releases/latest", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not a download url that will work with our system. I checked out your repo, and it doesn't seem like you have a publishing workflow in place. We require all plugins on the manifest to use an automated github workflow to build the plugin and attach it to a github release. So, please see the docs on the github workflow and add it accordingly. Once you're done with that, the direct url to the artifact on the release is the url that should go here.
Here is an example of a valid download url: https://github.com/mjtimblin/Flow.Launcher.Plugin.AwsToolkit/releases/download/v1.0.3/Flow.Launcher.Plugin.AwsToolkit.zip
"Author": "Rahul Gupta", | ||
"Version": "1.0", | ||
"Language": "python", | ||
"ExecuteFileName": "main.py", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This key is exclusive to you're plugin's plugin.json
file, and should not be provided here. See the plugin submission instructions for info about which keys you are supposed to provide here.
"Version": "1.0", | ||
"Language": "python", | ||
"ExecuteFileName": "main.py", | ||
"Keyword": "sp", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This key is exclusive to you're plugin's plugin.json
file, and should not be provided here. See the plugin submission instructions for info about which keys you are supposed to provide here.
Another thing to mention: We try to limit the about of duplicate plugins, and encourage developers to collaborate on similar plugins. The RecentProjectsOpen plugin seems to have the same functionality as this plugin, and though it supports numerous code editors, sublime isn't one of them. Since both plugins are written in python, I would suggest migrating from creating this plugin to contributing to the RecentProjectsOpen plugin by implementing support for sublime. |
You seem to have fixed most things, however your release artifact seems to have been uploaded manually, and your publish workflow seems to have failed. Please fix your workflow and have the workflow upload the artifact. Additionally, do you mind giving a reason as to why you have chosen to progress with the creation of this plugin instead of contributing to the RecentProjectsOpen plugin?
|
No description provided.