Skip to content

Ameliorations rochefort #20

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

Merged
merged 18 commits into from
Apr 7, 2025
Merged

Ameliorations rochefort #20

merged 18 commits into from
Apr 7, 2025

Conversation

adurand-cs
Copy link
Contributor

@adurand-cs adurand-cs commented Apr 4, 2025

Dans cette PR:

@thbar
Copy link
Collaborator

thbar commented Apr 4, 2025

@adurand-cs j'ai traité un conflit !

Est-ce que c'est OK pour reviewer et merger après ?

@thbar
Copy link
Collaborator

thbar commented Apr 5, 2025

@adurand-cs je continue sur la même branche ⚠️

@thbar
Copy link
Collaborator

thbar commented Apr 7, 2025

@adurand-cs c'est ok de mon côté ; j'ai ajouté le code qui permet de faire les petits résumés XML. Je vais aller le plugger dans une autre PR.

On relit de son côté et on valide et on merge ici !

@@ -0,0 +1,56 @@
# self-contained tests embryo
Copy link
Collaborator

Choose a reason for hiding this comment

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

Tests qui m'ont permis de mettre au point le petit outil de résumé XML.

elixir lib/test/wip_test.exs pour lancer.

Copy link
Collaborator

Choose a reason for hiding this comment

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

(ça me permet aussi de vérifier si on n'a pas trop cassé de code à côté).

@thbar thbar self-requested a review April 7, 2025 09:24
Copy link
Collaborator

@thbar thbar left a comment

Choose a reason for hiding this comment

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

Ça a l'air bon, la PR est grosse on fera plus petit aux prochains tours

<summary class="cursor-pointer font-bold p-4"></i>Solution</summary>
<div class="p-4">

```
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pourquoi ce formattage ? On ne peut pas juste afficher du texte simple ? Là ça donne un rendu "code" pour du texte

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah mince, moi à l'écran ça fait du markdown bien rendu normalement !!!

Je vais vérifier (mais là j'arrive à la gare ; donc on merge comme ça ou je corrige après)

@@ -664,7 +789,7 @@ Kino.Layout.tabs([
])
```

## DESCRIPTION DES PERIODES ET DES JOURS TYPES
## DESCRIPTION DES PERIODES ET DES JOURS TYPES - 01:10:00

🫵 À vous : ouvrez le fichier des calendriers
Copy link
Contributor Author

@adurand-cs adurand-cs Apr 7, 2025

Choose a reason for hiding this comment

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

Il faudra enlever cette ligne (A vous) : on a dit qu'on ne les faisait pas manipuler en même temps que nous

@thbar thbar merged commit b9f3551 into main Apr 7, 2025
@thbar thbar deleted the ameliorations-rochefort branch April 7, 2025 09:30
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