-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Create pet-the-capybara #5930
Create pet-the-capybara #5930
Conversation
New plugin |
Its my first time working with these checks, I'll close and retry with a fix for the error! |
Do not use the |
FYI you do not need to close this PR; just push updated commits which fix the errors, and we'll review it once it's passing CI |
Alright I'll keep trying, thanks a lot! (Sorry if this is giving notifications) |
This can be reduced to use |
I'd be up to improve the code anytime, its a big learning curve for me! |
It's generally best to clone core and check out some core plugins. Ctrl+shift+f in IJ |
Thanks for the feedback! MenuEntryAdded is now used instead of onMenuOpened and I made the code a bit more presentable. |
My first comment is still valid. |
Ooh I'll look into that tomorrow, I kinda have an idea what you mean but not completely! |
Your package should be called something like |
Simple plugin making it possible for users to pet a capybara and say a cute message on the client side.