Skip to content

Corrections diverses#13

Closed
Oumph wants to merge 0 commit into
phyver:masterfrom
Oumph:master
Closed

Corrections diverses#13
Oumph wants to merge 0 commit into
phyver:masterfrom
Oumph:master

Conversation

@Oumph
Copy link
Copy Markdown
Contributor

@Oumph Oumph commented Mar 27, 2021

  • ajout de bsdmainutils (pour la mission 12)
  • corrections orthographiques diverses (dont renommage d'une variable)
  • reformulation pour éviter les accords féminin/masculin
  • suppression de fichiers a priori inutiles (09_cp_1/._30_cp 10_cp_2/._31_cp 11_cp_3/._32_cp)

Copy link
Copy Markdown
Owner

@phyver phyver left a comment

Choose a reason for hiding this comment

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

Super !
Merci pour ce boulot.
Y'a juste quelques trucs que je vais laisser : "chaine", "maitriser", "connaitre" et "apparaitre". Si je ne me trompe pas, ces accents circomflexes ont disparus en 1990.

Objectif secondaire
===================

maitriser l'utilisation de la touche "Tabulation" pour
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

là, je garde "maitriser" !
:)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Zut, j'avais commentés tous les changements en question, mais j'ai du merdé lors du review final.
Désolé. (J'utilise assez peu github...)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Bah j'avais mal créé ma branche initiale (pour ne pas faire la PR depuis master/main), un partout :).

@Oumph Oumph closed this Mar 27, 2021
phyver pushed a commit that referenced this pull request Jul 13, 2022
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