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

Copy Ruby Track's core progression #1633

Merged
merged 16 commits into from
Dec 18, 2018
Merged

Conversation

emcoding
Copy link
Contributor

@emcoding emcoding commented Dec 16, 2018

This is to get you all started with the rearrangement of the track, as proposed in #1614
Please note that I don't know Python; this was done based on the changes I made to the Ruby track.
Hope it helps!

  1. Copy Ruby's core exercise progression as a first step to help decrease the mentoring workload.
  2. For easy reference, I put all the core exercises at the top of the file.
  3. Removed Leap as a core exercises, and choose other cores to unlock the side exercises that used to depend on Leap
  4. Raindrops was marked 'deprecated', but is part of the Ruby core; I reinstated it for now.
  5. In Ruby Difference of Squares and Grain are still cores, but will be changed to sides. As Python didn't have them as core, I left it like that.
  6. In Ruby, Robot Name is a core, but it is not an exercise in Python. To avoid extra work, I added Word Count instead. 00d5afa
  7. Extra cores that are in Python, not in Ruby:
    Bob, Allergies, Sum of Multiples, Kindergarten, Grade School, Saddle Points, BinarySearch, List ops, Bookstore, Markdown, Rest-api.
    I added commits for each of them, so that you can easily revert them. My changes are based on either considerations I made to the Ruby track, or a rough estimation in regard to the changes in this PR.
  • Kept as core: Kindergarten Garden; Grade School; Book Store; Markdown.
  • Changed into side: Bob, Allergies, Sum of Multiples, Saddle Points, Binary Search, List Ops; Rest-API.

Closes #1614

Maintainers To do :

  • Squash PR before merging
  • Ping Jeremy and ask to un-deprecate Python/Raindrops

- Copy Ruby's core exercise progression as a first step to help decrease the mentoring workload.
@emcoding emcoding changed the title Copy Ruby Track's core progression WIP Copy Ruby Track's core progression Dec 16, 2018
As Leap is not core anymore, the dependent side exercises need to be unlocked by something else. The choice is quite arbitrary.
Un-core, same as in Ruby.
Un-core, same as in Ruby: In Ruby we don't want bitwise operation exercises in core.
Un-core, because it's a math exercise
We don't have it (yet!) in the Ruby track, but I saw no reason why it couldn't be a core. 
I put it between somewhere midway.
Same as with Kindergarten: not a core in Ruby, but no obvious reason that it makes it unfit as a core exercise.
Make Saddle Points a side, because it's too similar to Matrix to have them both as a core.
Same as we did in Ruby: we made topics with CS and/or Data Structures a topic for side exercises.
.. because we want that in Ruby too ;-) But, because it's hard in Ruby, I moved it to the end of the track.
We're probably going to add it to the Ruby track as a core too.
Yes or no? Not quite sure. I'd expect it's better as a side exercise, but I had no good reason so I kept it for now, nearly to the end of the track.
There's a gap in the intermediate exercises, where in Ruby we have exercises that Python doesn't have. So, I promoted a few exercises that are on that same level in Ruby.
@emcoding emcoding changed the title WIP Copy Ruby Track's core progression Copy Ruby Track's core progression Dec 16, 2018
@cmccandless cmccandless self-assigned this Dec 17, 2018
config.json Outdated Show resolved Hide resolved
Copy link
Contributor

@cmccandless cmccandless left a comment

Choose a reason for hiding this comment

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

I've found this helpful when making major changes to config.json: https://jonmcalder.shinyapps.io/exercism-config-viz/

config.json Outdated
"uuid": "7961c852-c87a-44b0-b152-efea3ac8555c",
"slug": "rest-api",
"uuid": "2abe6eed-c7e4-4ad7-91e3-778bc5176726",
"core": true,
Copy link
Contributor

Choose a reason for hiding this comment

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

rest-api still has some sizable issues; it should probably be a side exercise for now. It might merit promotion when it is more developed.

Copy link
Contributor Author

@emcoding emcoding Dec 17, 2018

Choose a reason for hiding this comment

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

I agree it looks like it's better as a side exercise.

cmccandless and others added 3 commits December 17, 2018 17:11
take `"deprecated": false` out

Co-Authored-By: F3PiX <emcoding@pm.me>
@emcoding
Copy link
Contributor Author

@cmccandless Back to you. Made the changes. 🎉

@cmccandless
Copy link
Contributor

I think this looks good! As always, we may find the need for further tweaks down the road, but I think this should help with the difficulty curve on this track.

@cmccandless cmccandless merged commit addaaf1 into exercism:master Dec 18, 2018
@emcoding
Copy link
Contributor Author

Cool! Yeah, there will be tweaks needed, but I hope this helps.
Do I need to ping Jeremy to un-deprecate RainDrops? Or did you?

@emcoding emcoding deleted the patch-2 branch December 18, 2018 15:25
@cmccandless
Copy link
Contributor

Do I need to ping Jeremy to un-deprecate RainDrops?

@iHiD, would you kindly?

@iHiD
Copy link
Member

iHiD commented Dec 18, 2018

It seems to have magically done it itself! :)

@iHiD
Copy link
Member

iHiD commented Dec 18, 2018

Thank you both!

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.

3 participants