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

DEV-#### Review of subject of the first two quests of AI piscine #2737

Merged
merged 2 commits into from
Sep 26, 2024

Conversation

Frenchris
Copy link
Member

Why?

This piscine needed to be tested as a student.

Solution Overview

Fixes of inconsistencies, unclear instructions and grammar

@Frenchris Frenchris added the 📕 AI Artificial Intelligence label Sep 25, 2024
Copy link
Contributor

@Oumaimafisaoui Oumaimafisaoui left a comment

Choose a reason for hiding this comment

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

LGTM in term of instructions clarity changes and additional details, only prettier formatting would need to be checked again.

Copy link
Member

@MSilva95 MSilva95 left a comment

Choose a reason for hiding this comment

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

LGTM

@Oumaimafisaoui Oumaimafisaoui self-assigned this Sep 26, 2024
@Oumaimafisaoui Oumaimafisaoui merged commit 79d92e8 into master Sep 26, 2024
3 checks passed
@Oumaimafisaoui Oumaimafisaoui deleted the reviewAI.GO branch September 26, 2024 11:18
@HarryVasanth HarryVasanth added the 🗂️ docs Documentation label Sep 27, 2024
@HarryVasanth HarryVasanth changed the title [EXTERNAL] Review of subject of the first two quest of AI piscine DEV-#### Review of subject of the first two quests of AI piscine Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📕 AI Artificial Intelligence 🗂️ docs Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants