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

feat: validation codes geo et autres #7

Merged
merged 39 commits into from
May 28, 2024

Conversation

Sarrabah
Copy link
Collaborator

@Sarrabah Sarrabah commented May 2, 2024

Cette PR a pour but d'initialiser la validation de codes suivants:

*latitude_l93
*longitude_l93
*code_rna

@Sarrabah Sarrabah requested a review from pierrecamilleri May 2, 2024 10:41
@pierrecamilleri pierrecamilleri force-pushed the feat/Validation_codes_geo_et_autres branch from 595ca69 to 8bafd35 Compare May 3, 2024 08:56
Copy link
Collaborator

@pierrecamilleri pierrecamilleri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci !

En plus des points que j'ai soulevés, pour finaliser cette PR, il reste à :

  • factoriser CustomStrFormat et CustomFloatFormat
  • Rajouter la longitude (les deux vont souvent ensemble !)

EDIT : ah et un détail supplémentaire : j'ai fait un rebase + force push, donc n'oublies pas de récupérer la branche distante. La manière la plus simple à mon sens :

# Récupérer les changements distants
git fetch
# Supprimer la branche localement - attention de ne pas avoir de changements absents de la branche distante !
git branch -D feat/Validation_codes_geo_et_autres
# Recréer à partir de la branch distante
git switch feat/Validation_codes_geo_et_autres

.vscode/settings.json Outdated Show resolved Hide resolved
src/frformat/custom_format.py Show resolved Hide resolved
src/frformat/geo/latitude_l93.py Outdated Show resolved Hide resolved
src/tests/test_geo_fr.py Outdated Show resolved Hide resolved
@Sarrabah Sarrabah changed the title Feat: validation codes geo et autres feat: validation codes geo et autres May 3, 2024
@Sarrabah Sarrabah force-pushed the feat/Validation_codes_geo_et_autres branch from cff11e8 to 8b4b803 Compare May 4, 2024 17:25
@Sarrabah Sarrabah force-pushed the feat/Validation_codes_geo_et_autres branch from 8b4b803 to 9fb1b51 Compare May 4, 2024 17:28
src/frformat/code_rna.py Outdated Show resolved Hide resolved
src/frformat/code_rna.py Outdated Show resolved Hide resolved
src/frformat/code_rna.py Outdated Show resolved Hide resolved
src/frformat/code_rna.py Outdated Show resolved Hide resolved
src/frformat/geo/latitude_l93.py Outdated Show resolved Hide resolved
src/frformat/geo/longitude_l93.py Outdated Show resolved Hide resolved
src/frformat/geo/latitude_l93.py Outdated Show resolved Hide resolved
src/frformat/geo/longitude_l93.py Outdated Show resolved Hide resolved
src/tests/test_code_rna.py Outdated Show resolved Hide resolved
src/tests/test_code_rna.py Outdated Show resolved Hide resolved
src/tests/test_geo_fr.py Outdated Show resolved Hide resolved
src/tests/test_geo_fr.py Outdated Show resolved Hide resolved
src/frformat/code_rna.py Outdated Show resolved Hide resolved
src/frformat/geo/latitude_l93.py Outdated Show resolved Hide resolved
src/frformat/geo/longitude_l93.py Outdated Show resolved Hide resolved
src/frformat/geo/latitude_l93.py Outdated Show resolved Hide resolved
@pierrecamilleri pierrecamilleri marked this pull request as ready for review May 28, 2024 08:46
@pierrecamilleri pierrecamilleri merged commit 23df40b into main May 28, 2024
2 of 3 checks passed
@pierrecamilleri pierrecamilleri deleted the feat/Validation_codes_geo_et_autres branch May 28, 2024 08:47
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