Skip to content

Practice entries in config.json #960

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

Merged
merged 40 commits into from
Jun 16, 2025

Conversation

colinleach
Copy link
Contributor

With 25 concepts on the board, maybe it's time to add some more practice exercises. - or at least think about it This starter list isn't at all definitive, just something to start a conversation.

I've tried to pick out exercises which are both easy and fairly well focused on one or two concepts, rather than a whole-language workout.

Colin Leach added 30 commits April 2, 2025 15:14
@depial
Copy link
Contributor

depial commented Jun 13, 2025

Is it expected to have a non-zero intersection between practices and prerequisites?

@colinleach
Copy link
Contributor Author

I think so, but I'll check. There should be plenty of time for that - I'm far from confident that these are the best choice of exercises and concepts.

@depial
Copy link
Contributor

depial commented Jun 14, 2025

I think I misunderstood the structure... The concept should be finished first, which is the prerequisite, then the practice exercise practices the concept, right?

@colinleach
Copy link
Contributor Author

Err... I think so. Please don't assume I know what I'm talking about!

@depial
Copy link
Contributor

depial commented Jun 16, 2025

Another (somewhat related?) thing I could float here is to add Tuples to the prereqs for Composite Types.

I feel that the topics are related, and Tuples is just kinda hanging freely out there as a leaf. I'm not sure how you'd feel about that idea, though.

@colinleach
Copy link
Contributor Author

colinleach commented Jun 16, 2025

add Tuples to the prereqs for Composite Types

I was going to agree with this, then I looked more closely at the config.json and realized we have a different problem. In my head, and on the flowchart, Tuples is a prereq for functions, which is in turn a prereq for Types (and all that follows below). Somehow, we/I missed this off the entry for Locomotive Engineer.

{
        "slug": "locomotive-engineer",
        "name": "Locomotive Engineer",
        "uuid": "3341044c-6dd6-4a7b-a841-3a1215cd177b",
        "concepts": [
          "functions"
        ],
        "prerequisites": [
          "vector-operations",
          "pairs-and-dicts"
        ],
        "status": "beta"
 }

I think the better fix is to add tuples here, as it should long have been!

Another PR incoming...

@colinleach
Copy link
Contributor Author

You approved these. Does that mean you want them merged as-is?

@depial
Copy link
Contributor

depial commented Jun 16, 2025

If that's how the prereq structure should be, I'm fine with the selection you've made. So you can merge at will :)

@colinleach colinleach marked this pull request as ready for review June 16, 2025 22:30
@colinleach colinleach merged commit 88e6bc8 into exercism:main Jun 16, 2025
1 check passed
@colinleach colinleach deleted the practice_exercise_practices branch June 16, 2025 22:33
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