-
Notifications
You must be signed in to change notification settings - Fork 1k
document env variables in CLI reference #1719
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: main
Are you sure you want to change the base?
Conversation
🧪 testing bounty created!a testing bounty has been created for this PR: view testing issue testers will be awarded $20 each for providing quality test reports. please check the issue for testing requirements. |
@Omar8345 as already said by @neo773 This is not the Right Approach ! as it creates future maintenance burden, instead we should consolidate all the envs in a crate and reference it, then you can run codegen on the env.rs file to generate a markdown file. |
You can ask @neo773 cause its his Idea |
as per @neo773's approach/idea
@divanshu-go maybe leave your suggestions |
Feel free to work on this PR |
@Omar8345 Please resolve this mistake You have changed the permissions of these 2 files from please remove them from your PR as soon as possible chmod 755 screenpipe-vision/bin/ui_monitor
chmod 755 screenpipe-vision/bin/ui_monitor-aarch64-apple-darwin and then commit them |
@divanshu-go done |
@Omar8345 Also please Generate the Docs if your code can do that. |
@divanshu-go just run it like by cd to the folder |
@Omar8345 I'm just asking you to generate the docs and commit them. |
@divanshu-go All good, just one question, |
@Omar8345 while |
@Omar8345 You changed the formatting of |
oh god my prettifier thingy lmao |
@divanshu-go |
@Omar8345 yes ! |
@divanshu-go done ❤️ - sorry for the trouble as I'm not used to OSS contributing because I rarely do so & I'm new to screenpipe, love the software tho also do you have any idea if payouts to PayPal is possible since Algora needs Stripe & I don't have it - please confirm with me before proceeding with bounty |
@divanshu-go please let me know if there's something more |
@divanshu-go Fixed it, I used a comment in MDX ( |
@Omar8345 Why are doing this again and again ? |
@divanshu-go I believed I fixed it, here again now its confirmed to be fixed |
@divanshu-go @louis030195 please update |
Hi @Omar8345, Please understand that reviews can't be rushed or forced — we appreciate your patience. |
@louis030195 Please update. |
can I have some estimate please |
document all (as I believe) environment variables for screenpipe CLI in CLI reference documentation
/claim #1298