onlydustxyz / starklings

An interactive tutorial to get you up and running with Starknet
309 stars 83 forks source link

feat: add course displayer #198

Closed tdelabro closed 2 years ago

tdelabro commented 2 years ago
./dist/starklings/starklings --display-course
exercises/
    syntax/
      - syntax01
      - syntax02
      - syntax03
      - syntax04
      - syntax05
    strings/
      - strings00
      - strings01
    storage/
      - storage01
      - storage02
      - storage03
    implicit_arguments/
      - implicit_arguments01
      - implicit_arguments02
      - implicit_arguments03
    recursions/
      - recursion01
      - array01
      - array02
      - array03
      - array04
      - struct01
      - collatz_sequence
    registers/
      - registers00
      - registers01
      - registers02
      - registers03
    revoked_references/
      - revoked_references01
    tricks/
      - no_conditionals
      - assert_bool
    hints/
      - hints00
      - hints01
Bernardstanislas commented 2 years ago

Sure, what about "model.py" inside "exercises" folder then?

Le mer. 15 juin 2022 à 10:26, Timothée Delabrouille < @.***> a écrit :

@.**** commented on this pull request.

In src/exercises/init.py https://github.com/onlydustxyz/starklings/pull/198#discussion_r897686236 :

@@ -1,42 +1,86 @@ from src.config import exercises_directory, current_working_exercises_directory

+from .course_representation_types import Directory, Exercice

The thing is there is already a lot of stuffs named exercise in this project. Starting by the directory course_representation_types is in. In a rust project I will just call it types.rs, here I choose to be a bit more descriptive. But a file named exercise inside a directory named exercises doesn't add any useful information

— Reply to this email directly, view it on GitHub https://github.com/onlydustxyz/starklings/pull/198#discussion_r897686236, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABB23MOK2YHV7S4IFWRFY2LVPGHS5ANCNFSM5YYOZFHQ . You are receiving this because your review was requested.Message ID: @.***>