-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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 NinetyFive #9032
Add NinetyFive #9032
Conversation
Any feedback is appreciated |
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.
Automated testing result: WARNING
Repo link: NinetyFive
Results help
Packages added:
- NinetyFive
Processing package "NinetyFive"
- WARNING: The package does not contain a top-level LICENSE file. A license helps users to contribute to the package.
|
Updated! |
Be careful that you don't run into potential case sensitivity issues with your package name. Especially in file paths like those in your main menu file. But also elsewhere you're mixing NinetyFive and Ninetyfive. |
Thanks for catching this! I've made the updates. |
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.
Automated testing result: WARNING
Repo link: NinetyFive
Results help
Packages added:
- NinetyFive
Processing package "NinetyFive"
- WARNING: The package does not contain a top-level LICENSE file. A license helps users to contribute to the package.
My package is ...
My package is similar to other code assistants, however, it: