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

Ajout du préfixe "v" devant le numéro de version + intégration de "checks" automatisés #58

Merged
merged 3 commits into from
Apr 4, 2024

Conversation

thbar
Copy link
Contributor

@thbar thbar commented Apr 4, 2024

Je réalise ici un test sur ce qu'a implémenté @Pierlou :

EDIT: le test est concluant, j'ai corrigé le numéro de version, et on intègre ça pour la suite.

Voir:

@thbar
Copy link
Contributor Author

thbar commented Apr 4, 2024

@Pierlou j'ai ajouté "v" devant la version, est-ce que tu confirmes que cela fonctionnera bien sur les différents outils actuellement (schema.data.gouv.fr, validata etc) ?

@thbar thbar changed the title [WIP] Amélioration des checks automatiques Ajout du préfixe "v" devant le numéro de version + intégration de "checks" automatisés Apr 4, 2024
@thbar thbar requested a review from a team April 4, 2024 08:40
@thbar
Copy link
Contributor Author

thbar commented Apr 4, 2024

@etalab/transport-tech review davantage pour partager la connaissance là dessus que pour un vrai besoin de validation à ce stade !

@thbar thbar enabled auto-merge (squash) April 4, 2024 08:43
Copy link
Member

@AntoineAugusti AntoineAugusti left a comment

Choose a reason for hiding this comment

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

.github/workflows/assert_version.yml Outdated Show resolved Hide resolved
.github/workflows/assert_version.py Outdated Show resolved Hide resolved
.github/workflows/assert_version.py Show resolved Hide resolved
.github/workflows/assert_version.py Show resolved Hide resolved
@AntoineAugusti
Copy link
Member

Ah je fais une review ici alors que ça concerne plus l'autre dépôt, ça m'apprendra à lire la description. cc @Pierlou

@AntoineAugusti
Copy link
Member

J'aurais dû avoir la puce à l'oreille en voyant du Python 🐍 dans une PR de Thibaut 😏

@thbar
Copy link
Contributor Author

thbar commented Apr 4, 2024

Ah je fais une review ici alors que ça concerne plus l'autre dépôt, ça m'apprendra à lire la description. cc @Pierlou

Aucun souci, c'est en fait une bonne remarque dans le sens où @Pierlou on gagnerait à avoir une "GitHub action publiée" qu'on référence (comme le classique https://github.com/actions/checkout) dans chaque repo à terme, pour éviter le copier-collé, une fois que ça sera rodé !

J'aurais dû avoir la puce à l'oreille en voyant du Python 🐍 dans une PR de Thibaut 😏

Hihi :-)

@Pierlou
Copy link

Pierlou commented Apr 4, 2024

j'ai ajouté "v" devant la version, est-ce que tu confirmes que cela fonctionnera bien sur les différents outils actuellement (schema.data.gouv.fr, validata etc) ?

Oui ça se passe bien, c'est le cas notamment dans ce schéma

@thbar
Copy link
Contributor Author

thbar commented Apr 4, 2024

J'ai backporté les derniers changements apportés à datagouv/datapackage-template#1.

@thbar
Copy link
Contributor Author

thbar commented Apr 4, 2024

@AntoineAugusti @Pierlou à vot' bon coeur pour une revue finale (qui n'atterrira pas à ce stade dans une nouvelle version je précise).

Copy link
Member

@AntoineAugusti AntoineAugusti left a comment

Choose a reason for hiding this comment

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

Merci d'avoir repris ça.

Dommage pour la 500 côté AFIREV

@thbar thbar merged commit 801d198 into master Apr 4, 2024
2 checks passed
@thbar thbar deleted the version-testing-test branch April 4, 2024 13:58
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