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

Add a python library to access ods datas #12

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jvanoost
Copy link

@jvanoost jvanoost commented Aug 24, 2021

Des classes et scripts python pour accéder au portail ODS via l'API Manager

@fpassaniti
Copy link
Contributor

Hello @jvanoost

Merci bcp pour ce partage et initiative.
En toute transparence, on a pas spécialement la bande passante pour challenger, reprendre, review etc les propositions de ce type, par contre c'est une super source d'inspiration, point de départ pour d'autres qui voudraient scripter leur traitement sur leur plateforme, donc on veut garder ça.

On pensait faire un dossier contrib, à la racine du repo, avec un disclaimer "Contribution de la communauté etc..."
Qu'en penses tu ? le seul truc c'est qu'il faut trouver un moyen de revoir l'arbo vis à vis de ton fork... sinon je fais simple, je merge, je déplace, je commit.

Qu'en penses tu ?

@jvanoost
Copy link
Author

jvanoost commented Nov 8, 2022

Hello.

Intéressant l'idée des contributions. Ça peut en effet inspirer les autres.
Je pense aux usages des collègues rencontrés lors du club métropole en septembre, ça aurait bien sa place ici.

Je pense qu'en effet le plus simple pour toi c'est peu être de merger et tout réorganiser comme tu le sens.
S'il faut améliorer le readme de mon côté c'est possible aussi 😉

En tout cas moi ces bibliothèques je les utilise tous les jours sur différents scripts qui tournent en nocturne sur un serveur chez nous.
C'est toujours améliorable par qui veut bien sûr parce que c'est sûrement pas parfait mais si ça peut éviter à d'autres de repartir d'une feuille blanche...

++

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.

2 participants