-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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 plugin: Infio Copilot #5083
base: master
Are you sure you want to change the base?
Conversation
Thank you for your submission, an automated scan of your plugin code's revealed the following issues: Required[1]:Please remove the [1][2][3][4][5][6][7][8][9][10][11][12]:You should not cast this, instead use a [1][2][3][4][5][6][7][8][9][10][11]:You should avoid assigning styles via JavaScript or in HTML and instead move all these styles into CSS so that they are more easily adaptable by themes and snippets. [1][2]:We recommend against providing a default hotkey when possible. The reason being that it's easy to pick a hotkey that a user already has configured and they could get confused when that key doesn't do what they expect. Also, it's hard choosing a safe default hotkey that's available for all operating systems. [1][2][3][4][5][6][7][8][9][10][11]:You should consider limiting the number of Optional[1][2]:Casting to Do NOT open a new PR for re-validation. |
/skip
These values change dynamically during user interaction and must be calculated in JavaScript to ensure the menu appears in the correct position relative to the text being edited. |
I have modified everything except for the unchangeable CSS. Could you please review again? I have released version 0.0.3. |
How should I now reinitiate the review process? |
I have released version 0.04, which addresses the aforementioned issues. Could you please review it again? Thank you very much. https://github.com/infiolab/infio-copilot/releases/tag/0.0.4 |
Hello!I found the following issues in your plugin submission Errors: ❌ The newly added entry is not at the end, or you are submitting on someone else's behalf. The last plugin in the list is: This check was done automatically. Do NOT open a new PR for re-validation. Instead, to trigger this check again, make a change to your PR and wait a few minutes, or close and re-open it. |
In case of conflict, move my description to the end of the list.
Hello!I found the following issues in your plugin submission Errors: ❌ Could not parse This check was done automatically. Do NOT open a new PR for re-validation. Instead, to trigger this check again, make a change to your PR and wait a few minutes, or close and re-open it. |
I am submitting a new Community Plugin
Repo URL
Link to my plugin: https://github.com/infiolab/infio-copilot
Release Checklist
main.js
manifest.json
styles.css
(optional)v
)id
in mymanifest.json
matches theid
in thecommunity-plugins.json
file.I have given proper attribution to these other projects in my
README.md
.