Skip to content

Add a Python template#2

Open
dmerejkowsky wants to merge 13 commits intoarolla:mainfrom
dmerejkowsky:feature-python-template
Open

Add a Python template#2
dmerejkowsky wants to merge 13 commits intoarolla:mainfrom
dmerejkowsky:feature-python-template

Conversation

@dmerejkowsky
Copy link
Copy Markdown

Based on work by Pierre-Yves Pamart in #1

There are several things that can be improved, let' s use this PR to work on it collectivley with the Arolla Python Guilde.

- Add .gitignore

- Add requirements.txt

- Add "add" function

- Add pytest config

- Add feature file

- Add steps

- Add README
@dmerejkowsky
Copy link
Copy Markdown
Author

dmerejkowsky commented Oct 24, 2022

Raisons pour simplifier le .gitignore

1/ C'est en général une mauvaise idée de versionner des fichiers auto-générés

2/ Le fichier est trop gros et ça rend son édition compliquée

3/ C'est facile à modifier au cas par cas

4/ En l'état c'est facile de pas comprendre pourquoi un git add ne fonctionne pas

@dmerejkowsky
Copy link
Copy Markdown
Author

dmerejkowsky commented Oct 24, 2022

Tâches restantes:

  • requirements.txt ne devrait pas être un fichier binaire
  • revoir le fichier pytest.ini - peut-on s'en passer ?
  • tests/step : réduire la duplication entre initialize_first_number_equal_to_50 et initialize_second_number_equal_to_70
  • pareil pour les deux scénarios : choisir entre "templaté avec un tableau" ou tout hard-codé
  • Supprimer les blocs de commentaires ################
  • Détection automatique des scénarios : from pytest_bdd import scenarios ; scenarios("features")
  • Choisir entre "supprimer toutes les annotations de types" ou "tout annoter" - je préfère sans annotations du tout pour les débutants
  • Formatter le code avec black

@pypamart
Copy link
Copy Markdown

Après vérification le fichier requirements.txt est bel et bien en txt et non en binaire, bug affichage Github ?

Le pytest.ini est optionnel mais te permets de juste saisir "pytest" pour lancer les tests, et sert aussi pour de la customisation de l'affichage des messages.

Oui, tout est formaté avec Black.

Pour le reste c'est votre parti pris que je respecte mais sans être en accord ;-)

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