Skip to content
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

fix: correct indentation for macOS activation policy in main.rs #1101

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zelosleone
Copy link


name: [pull request](fix: correct indentation for macOS activation policy in main.rs)
about: removed the second cfg macos, one is enough to use it correctly
title: "[pr] "
labels: ''
assignees: ''


description

just removed the second cfg macos issue, now window is building on github actions just fine and test results on windows are good as well. unless you meant the ollama build, which still builds and has a long time to run to be honest.

/claim #1100

related issue: #1100

how to test

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

  1. tested it via github actions on my own repo/branch

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

would only take 5 minutes at most to be honest.

Copy link

vercel bot commented Jan 8, 2025

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 Jan 8, 2025 0:50am

@louis030195
Copy link
Collaborator

can you share video of app working

@zelosleone
Copy link
Author

can you share video of app working

Github CI/CL working?

@louis030195
Copy link
Collaborator

i dont get what this PR is doing tbh, build works

https://github.com/mediar-ai/screenpipe/actions/runs/12663602032/job/35291164994

@louis030195 louis030195 marked this pull request as draft January 9, 2025 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants