Skip to content

Core exercises (Round1)#83

Merged
Average-user merged 6 commits intoexercism:masterfrom
Average-user:core-exercises
Apr 7, 2019
Merged

Core exercises (Round1)#83
Average-user merged 6 commits intoexercism:masterfrom
Average-user:core-exercises

Conversation

@Average-user
Copy link
Copy Markdown
Member

No description provided.

@Average-user
Copy link
Copy Markdown
Member Author

Trying to get the Prolog track structured following Round 1 of project track anatomy. This are only early steps, I presume that with following rounds this could change a lot.

@Average-user
Copy link
Copy Markdown
Member Author

Current order of core exercises:

  • hello-world
  • queen-attack
  • triangle
  • isogram
  • binary
  • rna-transcription
  • anagram
  • pascals-triangle
  • dominoes

@emcoding
Copy link
Copy Markdown

@Average-user I just realized that all the side exercises are still unlocked_by: null, meaning that they will be presented to the student immediately after Hello World.

To prevent this, they should be unlocked by one of the core exercises.

You probably want to add that to this PR.

@Average-user
Copy link
Copy Markdown
Member Author

@F3PiX Isn't that part of some other Round in the tool?

@emcoding
Copy link
Copy Markdown

emcoding commented Mar 18, 2019

Usually, yes. But in case of Prolog, before this PR all the exercises were core: false, unlocked_by: null. (And the website randomly picked some to appear as 'core' to students.)

Since this PR introduces real core exercises (core: true, unlocked_by: null), all other exercise are unlocked_by: null, meaning: will be 'unlocked' by Hello World. That's the opposite from what you try to accomplish with this PR.

I didn't realize this would be a problem before. This is the first track tested that had no real core exercises. I added the use case to the Tool, so this will be clear for next tracks in this situation.

@emcoding
Copy link
Copy Markdown

@Average-user How about merging this step first, and open an issue for redividing the side exercises?

There's not much profit in letting this PR sit here instead of merging :-)

@Average-user Average-user merged commit 4f3d852 into exercism:master Apr 7, 2019
@Average-user
Copy link
Copy Markdown
Member Author

@F3PiX Thanks for the help

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