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

MàJ du readme et archivage du contenu de l'ancien #135

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

TuThoThai
Copy link
Collaborator

Cette PR répond directement au ticket #70
J'ai essayé de faire de telle sorte que le texte colle à celui que j'ai proposé pour le répertpoire du profil France de SIRI.

@TuThoThai TuThoThai added the documentation Improvements or additions to documentation label Jan 19, 2025
@TuThoThai TuThoThai requested review from thbar and prhod January 19, 2025 17:54
Copy link
Collaborator

@prhod prhod left a comment

Choose a reason for hiding this comment

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

C'est bon pour moi, à part qques typo et une remarque de vocabulaire (j'attends qu'on statue pour approuver la PR)
Pourquoi garder l'ancien Readme à côté avec le nom "old README.md" ?

## Conversion .doc -> .md
Ces étapes ont permis la conversion des fichiers .doc vers les .md correspondants.
Il pourra être utile de s'y référer par la suite pour la conversion de nouveaux documents.
# Gouvernance
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ce chapitre parle de "répertoire" comme traduction de "repository", ce qui peut porter à confusion avec les "dossiers".
Le terme habituel en français est plutôt "dépôt", mais ça ne parlera pas au grand public. Je propose "espace de travail" comme alternative, mais sans grande conviction.
Ce point peut être ignoré s'il n'y a pas d'autres personnes gêné par la formulation.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

J'avais effectivement mis '' répertoire '' en traduction de '' repository ''. C'est également le choix dans le readme que j'ai proposé pour le repos du profil France de SIRI. Je suggère qu'on garde comme ça jusqu'à trouver mieux et changer les deux en même temps.

README.md Outdated Show resolved Hide resolved
Co-authored-by: Pascal Rhod <[email protected]>
@TuThoThai
Copy link
Collaborator Author

J'ai répondu pour le vocabulaire.
Pour l'ancien readme, c'est surtout pour éviter à l'équipe du PAN, notamment @thbar, de perdre 'euro historique de méthode. 😊

Copy link
Collaborator

@prhod prhod left a comment

Choose a reason for hiding this comment

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

Merci pour les retours, c'est bon pour moi.
Pour le fichier, il est dans l'historique Git, donc pas besoin de l'ajouter à côté. Mais je laisse @thbar confirmer le point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants