arcana/arkheon
0
0
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity Actions

fix: use sqlite as the only possible database backend #8

Merged
sinavir merged 1 commit from sqlite into main 2025-03-29 21:26:53 +00:00
Conversation 9 Commits 1 Files changed 2 +4 -16
sinavir commented 2025-03-26 14:07:15 +00:00 (Migrated from github.com)
Copy link
No description provided.
Tom-Hubrecht (Migrated from github.com) reviewed 2025-03-26 14:32:10 +00:00
module.nix
Tom-Hubrecht (Migrated from github.com) commented 2025-03-26 14:31:04 +00:00
Copy link

Y'a un / en trop non ?

Y'a un / en trop non ?
src/api/db.py
Tom-Hubrecht (Migrated from github.com) commented 2025-03-26 14:31:41 +00:00
Copy link

Ça change quoi de pas mettre BigInteger ?

Ça change quoi de pas mettre BigInteger ?
sinavir (Migrated from github.com) reviewed 2025-03-26 15:00:22 +00:00
src/api/db.py
sinavir (Migrated from github.com) commented 2025-03-26 15:00:22 +00:00
Copy link

Rien en terme de taille des entiers, sqlite ne connait juste pas bigint du coup ça fait de la merde

Rien en terme de taille des entiers, sqlite ne connait juste pas bigint du coup ça fait de la merde
sinavir (Migrated from github.com) reviewed 2025-03-26 15:01:52 +00:00
module.nix
sinavir (Migrated from github.com) commented 2025-03-26 15:01:52 +00:00
Copy link

Non ça marche pas sinon (pour une raison obscure que j'ai pas investiguée, pour tester tu peux checkout https://github.com/sinavir/arkheon/tree/update et utiliser le nixos test)

Non ça marche pas sinon (pour une raison obscure que j'ai pas investiguée, pour tester tu peux checkout https://github.com/sinavir/arkheon/tree/update et utiliser le nixos test)
Tom-Hubrecht (Migrated from github.com) reviewed 2025-03-26 16:01:43 +00:00
module.nix
Tom-Hubrecht (Migrated from github.com) commented 2025-03-26 16:01:43 +00:00
Copy link

Indeed, c'est dans la doc https://docs.sqlalchemy.org/en/20/core/engines.html#sqlite

Indeed, c'est dans la doc https://docs.sqlalchemy.org/en/20/core/engines.html#sqlite
sinavir (Migrated from github.com) reviewed 2025-03-26 16:08:24 +00:00
module.nix
sinavir (Migrated from github.com) commented 2025-03-26 16:08:24 +00:00
Copy link

J'ai menti

J'ai menti
Tom-Hubrecht (Migrated from github.com) reviewed 2025-03-26 16:09:40 +00:00
module.nix
Tom-Hubrecht (Migrated from github.com) commented 2025-03-26 16:09:40 +00:00
Copy link

Bah non ?
Si tu mets que 3 / ça considère que c'est un relative path

Bah non ? Si tu mets que 3 / ça considère que c'est un relative path
sinavir (Migrated from github.com) reviewed 2025-03-26 16:12:14 +00:00
module.nix
sinavir (Migrated from github.com) commented 2025-03-26 16:12:14 +00:00
Copy link

Ok juste je pensais mon test arrivait à tester ça mais visiblement non

Ok juste je pensais mon test arrivait à tester ça mais visiblement non
sinavir (Migrated from github.com) reviewed 2025-03-26 17:18:51 +00:00
module.nix
sinavir (Migrated from github.com) commented 2025-03-26 17:18:51 +00:00
Copy link

Du coup pour le fin mot de l'histoir le working directory est / ducoup mes tests passent évidement

Du coup pour le fin mot de l'histoir le working directory est / ducoup mes tests passent évidement
Tom-Hubrecht (Migrated from github.com) approved these changes 2025-03-29 21:26:32 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Tom-Hubrecht
Labels
Clear labels   
bug

Something isn't working

  
documentation

Improvements or additions to documentation

  
duplicate

This issue or pull request already exists

  
enhancement

New feature or request

  
good first issue

Good for newcomers

  
help wanted

Extra attention is needed

  
invalid

This doesn't seem right

  
question

Further information is requested

  
wontfix

This will not be worked on

No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
arcana/arkheon!8
No description provided.