Skip to content
Snippets Groups Projects

WIP: Resolve "counter test module"

Closed Ghost User requested to merge counter/117-counter-test-module into dev

Closes #117 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
33 33 #[cfg(unix)]
34 34 pub use durs_tui::TuiModule;
35 35 //pub use durs_skeleton::SkeletonModule;
36 pub use durs_counter::CounterModule;//ais-je bien défini durs_counter?
  • Ghost User added 5 commits

    added 5 commits

    Compare with previous version

  • Author Contributor

    Je reconnais que j'aurais dû automatiquement utiliser fmt AVANT de commiter le code. J'ai décidé de rester cohérent et j'ai pushé du code avec un commit [ref] pour formater le code que j'aurais dû formater avant. @librelois

  • @counter-reverse voici ma review :

    commit 1: Le module se nomme toujours skeleton dans le code, il te faut renommer le module dans ce 1er commit en remplaçant toutes les occurrences a Skeleton par un nom approprié. De plus lorsque l'on créer une nouvelle crate dans le projet il faut la déclarée dans le Cargo.toml a la racine du projet.

    commit 2 : Tu y a commité un fichier dont les changements n'ont aucun rapport avec le titre du commit, ces changements qui concernent le renommage du module doivent être déplacés dans le 1er commit.

    commit 3 : On ne fait jamais de commit spécial fmt, sauf lors d'une mise a jours de fmt ou l'on applique directement, sur dev la dernière version de fmt, tu n'est pas dans ce cas la. Il te faut revenir dans chacun de tes commit appliquer fmt puis amend (tu peut revenir au commit de ton choix avec l'option edit du rebase interactif :)

  • Ghost User added 3 commits

    added 3 commits

    • e4d5eca7 - [feat] create counter module
    • 2b506388 - [ref] counter module: added in the main Cargo.toml
    • 8c8e69a6 - [ref] counter: rename skeleton variables -> counter

    Compare with previous version

  • Author Contributor

    @librelois J'ai modifié le nom des variables dans le premier commit de la première push request. J'espère que c'était ce que tu voulais.

  • @counter-reverse voici ma review :

    commit 1 : Tu a commité des fichiers d'une crate ws2p ?!? Ces fichiers n'ont rien a faire dans ce commit, il faut les supprimer. Aussi bouge l'insertion dans durs-server dans un autre commit comme tu l'avais fait au départ. commit 2 : il faut le squasher avec le commit 1, dans un crer une nouvelle crate il faut la déclarée dans le Carto.toml a la racine, dans le commit ou al crate est créer. commit 3: a sa place dans le 1er commit

    Tu ne découpe pas encore de façon pertinente (c'est normal il faut du temps), ici un bon découpage atomique c'et :

    1. création de ton module vide (donc avec renomage de toute's les occurences a skeleton).
    2. Injection de topn module dans durs-server

    J'aimerais que tu arrive a traiter correctement mes retours avant de passer a la suite :)

  • Author Contributor

    @librelois étrange je n'ai jamais eu l'impression d'avoir touché à ws2p.

  • Author Contributor

    @librelois la commande git diff semble indiquer que je n'ai pas modifié le fichier ws2p dans aucun des trois commit pushés.

    Peut-être que j'ai du mal rebaser.

  • La preuve en image :

    image

    Ton 1er commit inclus la modification de 3 fichiers dans ws2p-v1, c'est parce que ta branche n'est pas rebasée sur dev (en tout cas sur le remote, c'est la seule chose que je peut voir je ne peut pas voir ta branche locale, si tu a bien rebaser ta branche locale alors il te faut push force pour que je puisse voir ça).

  • Author Contributor

    Merci. Je vais creuser.

  • Ghost User added 36 commits

    added 36 commits

    Compare with previous version

  • Author Contributor

    @librelois : la création du module et de son importation sont maintenant séparés dans deux commits différents.

    j'en ai profité pour mettre mon module counter à jour.

  • @counter-reverse bien tu progresse mais ce n'est pas encore ça au niveau du nommage du 2ème commit : la portée c'est d'abord nom de la crate et ensuite éventuellemetn le nom du sous module dans la crate voir du fichier si c'est un très gros sous module, mais ici ce n'est pas la cas. De plus main.rs n'est pas parlant car toutes les crates binaires ont un main.rs, ici la crate binaire a laquelle tu ajoute ton module c'estdurs-server, al seule existante a ce jours, mais a l'avenir il y aura aussi un durs-desktop (appli de bureau avec fenêtre) voir d'autres crates binaires encore (des nœuds spécialisés).

    Ensuite dans ton 1er commit tu modifie des fichiers de la crate durs-server, or ça ne devrait pas être le cas car dans ce commit la tu créer ton module et c'est tout, durs-server n'est pas encore impliqué a ce moment la.

    image

    Renommer correctement ton premier commit et déplace les diff de durs-server dans le 2ème et ce sera enfin ok :)

  • Author Contributor

    @librelois : Parlant des fichiers de la crate durs-server que je n'aurais pas du modifier dans le premier commit, je pense que j'ai du faire une rebase de la dernière version de la branche dev, modifiant malencontreusement les fichiers.

  • @counter-reverse c'est lr genre de truc a vérifier sur ta GUI git avant de commiter, person je vérifie toujours su mon kraken la liste des fichiers impactés par mon commit avant de commiter. Tu peut faire un rebase interactif avec le mot clés "edit" pour modifier ton 1er commit, il te suffira ensuite d'amend le 2ème.

    Tu peut aussi reset soft tes 2 commits puis recommiter tes fichiers bien séparément, il y a plusieurs façon de faire l'important c'est que in finé la séparation soi propre dans l'historique :)

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading