-
Notifications
You must be signed in to change notification settings - Fork 99
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
Publish to conda #1786
Publish to conda #1786
Conversation
@benoit-cty : me confirmes-tu qu'il n'y a plus de problème de longueur de chemin ? |
Oui et non : On ne peut pas intégrer OF-Fr à conda-forge à cause du problème de chemin (voir issue ) mais on peut le pousser vers le channel conda OpenFisca.org en ajoutant un paramètre de build pour builder dans un chemin court au lieu du chemin par défaut. Et j'ai testé sur mon Windows, l'installation fonctionne. Mais mon user est un nom court : "ben", ça joue surement. |
Dans
|
17b7670
to
b897cb9
Compare
@benjello : Désolé j'ai intégré ta review manuellement sans accepter ton commit, peux-tu revoir ta review ? Peut-être que je vais finalement réussir à être sur conda-forge également : conda-forge/staged-recipes#17483 Les deux ne sont pas incompatibles : on peut avoir toutes les versions en automatique sur le channel openfisca et les majeure seulement sur conda-forge car c'est une étape manuelles. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci @benoit-cty 🙌
Voici quelques retours (dont certains déjà transmis à l'oral mais notés ici pour partage du contexte).
Prise en compte de la review de @sandcha terminée, j'ai mis à jour la description de la PR pour résumer ce qu'elle contient. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Super travail.
La doc est claire et complète.
Je ne suis pas expert sur les autres aspects.
Et il y a une petite correction d'accord.
@sandcha J'ai vérifié que le package PyPi se génère toujours bien et que le readme apparaît maintenant bien sur PyPi : https://test.pypi.org/project/OpenFisca-France/104.0.3/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est top ! Merci @benoit-cty 🎉
Mini retex sur les opérations conda : en créant un nouvel environnement (ce n'était pas le premier), il a de nouveau fallu exécuter le |
f264b14
to
c90d271
Compare
Co-authored-by: sandcha <sandcha@users.noreply.github.com>
Co-authored-by: sandcha <sandcha@users.noreply.github.com>
Co-authored-by: sandcha <sandcha@users.noreply.github.com>
Co-authored-by: sandcha <sandcha@users.noreply.github.com>
Co-authored-by: sandcha <sandcha@users.noreply.github.com>
Co-authored-by: sandcha <sandcha@users.noreply.github.com>
Co-authored-by: sandcha <sandcha@users.noreply.github.com>
51be44b
to
ddac27c
Compare
Ces changements (effacez les lignes ne correspondant pas à votre cas) :
En détail:
Quelques conseils à prendre en compte :
setup.py
.CHANGELOG.md
.Et surtout, n'hésitez pas à demander de l'aide ! :)