Melora Hugues faercol · she/they
  • Joined on 2022-10-08
faercol created branch epic-10-ui in PolyculeConnect/polycule-connect 2023-10-22 12:47:42 +00:00
faercol pushed to main at PolyculeConnect/polycule-connect 2023-10-22 12:26:25 +00:00
e00347d96f fix: correct ci definition for woodpecker
a82c12fc93 chore: add vscode to gitignore
554d9bccc4 chore: drop drone support
Compare 3 commits »
faercol deleted branch fix/woodpecked-creds from PolyculeConnect/polycule-connect 2023-10-22 12:26:25 +00:00
faercol merged pull request PolyculeConnect/polycule-connect#6 2023-10-22 12:26:24 +00:00
Fix woodpecker config
faercol pushed to fix/woodpecked-creds at PolyculeConnect/polycule-connect 2023-10-22 12:18:15 +00:00
e00347d96f fix: correct ci definition for woodpecker
a82c12fc93 chore: add vscode to gitignore
554d9bccc4 chore: drop drone support
ccb417ad6a feat #4: add storage config in config file
Compare 4 commits »
faercol pushed to fix/woodpecked-creds at PolyculeConnect/polycule-connect 2023-10-22 12:18:06 +00:00
9b84dea234 fix: correct ci definition for woodpecker
faercol pushed to fix/woodpecked-creds at PolyculeConnect/polycule-connect 2023-10-22 12:09:49 +00:00
1959538971 fix: correct ci definition for woodpecker
b87a1cfbea chore: add vscode to gitignore
Compare 2 commits »
faercol pushed to fix/woodpecked-creds at PolyculeConnect/polycule-connect 2023-10-22 12:05:38 +00:00
bbea68443f chore: add vscode to gitignore
5e9884ccad fix: correct ci definition for woodpecker
Compare 2 commits »
faercol commented on pull request PolyculeConnect/polycule-connect#3 2023-10-22 11:34:18 +00:00
feat #4: add storage config in config file

En vrai ça me parait un bon cas d'usage pour une enum. C'est un peu plus lourd à définir, mais ça permettra de typer ce switch de façon plus précise (même si contrairement à Rust n'as pas de vérification d'exhaustivité)

faercol commented on pull request PolyculeConnect/polycule-connect#3 2023-10-22 11:34:18 +00:00
feat #4: add storage config in config file

Je l'aurais juste appelé config, pas besoin de plus, et ça évite les répétitions

faercol commented on pull request PolyculeConnect/polycule-connect#3 2023-10-22 11:34:18 +00:00
feat #4: add storage config in config file

pourquoi pas appeler ça juste StorageConfig? Y'a pas de dépendances dans le code je crois ?

faercol commented on pull request PolyculeConnect/polycule-connect#3 2023-10-22 11:34:18 +00:00
feat #4: add storage config in config file

En vrai vu qu'on va commencer à ajouter des backends ici, ça serait ptetre pertinent d'ajouter une fonction séparée pour initialiser le storage, ça permet d'éviter de trop alourdir la fonction main

faercol pushed to fix/woodpecked-creds at PolyculeConnect/polycule-connect 2023-10-22 10:06:12 +00:00
c8bbd4f260 fix: correctly use secrets for woodpecker
faercol pushed to fix/woodpecked-creds at PolyculeConnect/polycule-connect 2023-10-22 09:25:04 +00:00
7cb68c2de4 fix: correctly use secrets for woodpecker
faercol created pull request PolyculeConnect/polycule-connect#6 2023-10-22 09:13:24 +00:00
WIP: fix/woodpecked-creds
faercol pushed to fix/woodpecked-creds at PolyculeConnect/polycule-connect 2023-10-22 09:12:59 +00:00
e60003d44b fix: correctly use secrets for woodpecker
39ed7aa07f chore: drop drone support
Compare 2 commits »
faercol created branch fix/woodpecked-creds in PolyculeConnect/polycule-connect 2023-10-22 09:12:59 +00:00