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

Add more generics instructions #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

GregoireHebert
Copy link

For a non FrontEnd / non web public.

@GregoireHebert GregoireHebert added RFR Ready For Review enhancement New feature or request labels Feb 8, 2023
@GregoireHebert GregoireHebert self-assigned this Feb 8, 2023
- Enregistrez ce commit
- Créez une branche dédiée à votre fonctionnalité `feat/something-new`
- Réaliser l'une des modifications suivantes :
- Dans le fichier `index.html`:
Copy link
Contributor

@Gregcop1 Gregcop1 Feb 10, 2023

Choose a reason for hiding this comment

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

Ça, ça ne marche plus s'ils se sont basé sur un dépôt local plutôt que cloner le projet. Est-ce qu'il ne faudrait pas carrément supprimer le côté HTML dans ce cas, ça serait sans doute moins confusant ?

- Ajoutez de nouvelle modification au sein des premières lignes du fichier.
- Commitez ce changement toujours **sans la balise `section`** ou **sans le chien**.
- Mince ! Vous vous rendez-compte que ce changement n'aurait pas dû apparaître dans cette branche. Supprimez le commit.
- Créez une branche dédiée au fix `fix/my-correction` (c'est une correction de main, donc attention à la source de la branche utilisée) et appliquez-lui le commit que vous venez de supprimer.
Copy link
Contributor

Choose a reason for hiding this comment

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

J'imagine que tu y as pensé, mais il me semble qu'il faut modifier les slides pour correspondre à ces noms de branch quand ça sera mergé.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request RFR Ready For Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants