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

Testing - simplification #21

Merged
merged 23 commits into from
Apr 3, 2024
Merged

Testing - simplification #21

merged 23 commits into from
Apr 3, 2024

Conversation

Salamandar
Copy link
Member

  • toml styling
  • Simplify retrieval of manifest.toml info

@ewilly Is there a real reason why you manually install postgresql-v16 ?

@Salamandar
Copy link
Member Author

!testme

@yunohost-bot
Copy link
Contributor

Meow 🐈
Test Badge

@yunohost-bot
Copy link
Contributor

🪱
Test Badge

@ewilly
Copy link
Member

ewilly commented Apr 1, 2024

@ewilly Is there a real reason why you manually install postgresql-v16 ?

pgvector is required and is not in bullseye (nor bookworm) but will come with trixie => due to pgvector we need to use the postgresl repo for now.
V15 works too, the current dockercompose use v14 and trixie will come with v16 which is the current stable version according to postgresql website => so i go to V16 directly (i didn't test V14 or V13 from postegresql repo)

@ewilly
Copy link
Member

ewilly commented Apr 1, 2024

!testme

@yunohost-bot
Copy link
Contributor

🚀
Test Badge

@yunohost-bot
Copy link
Contributor

📚
Test Badge

@ewilly
Copy link
Member

ewilly commented Apr 1, 2024

!testme

@yunohost-bot
Copy link
Contributor

😜
Test Badge

@yunohost-bot
Copy link
Contributor

📚 🪱
Test Badge

@ewilly
Copy link
Member

ewilly commented Apr 1, 2024

!testme

@yunohost-bot
Copy link
Contributor

Fingers crossed!
Test Badge

@yunohost-bot
Copy link
Contributor

📚 🐛
Test Badge

@ewilly
Copy link
Member

ewilly commented Apr 2, 2024

!testme

@yunohost-bot
Copy link
Contributor

📚
Test Badge

@yunohost-bot
Copy link
Contributor

Alrighty!
Test Badge

@Salamandar
Copy link
Member Author

FYI @ewilly you don't need to re-run testme every time. Just click on the "results screenshots" and restart the jobs ;)

@Salamandar
Copy link
Member Author

Also: I can confirm this branch runs fine on my server (at least commit af79). I just don't have any face detection/recognition working, but that's unrelated.

@ewilly ewilly merged commit 9c8e0fa into master Apr 3, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants