-
Notifications
You must be signed in to change notification settings - Fork 27
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
New install guide and setup script #32
base: main
Are you sure you want to change the base?
Conversation
haha thanks for all the work, i knew my readme was really bad :) i'll edit some things and hopefully push tomorrow |
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.
i still say keep the installation in the wiki, or make an INSTALL.md file so we can go more in depth. putting the install with vague commands in the README.md feels off to me, but it's easily changeable
also, this is just preference, don't use copilot for this :) human made usually means human readable
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.
as for ble spam, i will work on this once i have a python package that isn't depreciated, pybluez seems an okay fit for this but we will see
also, please please please try to prevent running other processes without an absolute directory, this can be used for a privesc path traversal attack and i'd rather be safe than sorry
plus whats with the tons of added pypy packages?
it was for the ble spam, took it from applejuice |
Added How to install in the Readme and a setup script, the wiki is a bombshell.