fix: kill llm & screenpipe sidecar berfore updating on windows

tribhuwan-kumar commented 2 weeks ago

name: pull request about: kill llm and screepipe process before updating. but this issue #575 might occur again if user explicitly restarts the screenpipe backend from status ui while downloading the update, maybe i need to disable those button while update is being downloaded.

fix: #575

description

brief description of the changes in this pr.

related issue: #

type of change

[x] bug fix
[ ] new feature
[ ] breaking change
[ ] documentation update

how to test

add a few steps to test the pr in the most time efficient way.

if relevant add screenshots or screen captures to prove that this PR works to save us time.

checklist

[ ] MOST IMPORTANT: this PR will require less than 30 min to review, merge, and release to production and not crash in the hand of thousands of users
[ ] i have read the CONTRIBUTING.md file
[ ] i have updated the documentation if necessary
[ ] my changes generate no new warnings
[ ] i have added tests that prove my fix is effective or that my feature works

additional notes

any other relevant information about the pr.

vercel[bot] commented 2 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
screenpipe	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 5, 2024 10:46am

louis030195 commented 2 weeks ago

thanks :)

/tip $30 @tribhuwan-kumar

algora-pbc[bot] commented 2 weeks ago

🎉🎈 @tribhuwan-kumar has been awarded $30! 🎈🎊

mediar-ai / screenpipe