-
-
Notifications
You must be signed in to change notification settings - Fork 544
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
Retree: Rename retree to satellite #1451
Comments
@iHiD I like this idea a lot. I think it makes the exercises sound fun and intriguing and I like that the learner discovers the concepts as they're working through the exercise rather than extrapolating from the exercise name. |
Is fine by me. |
Presuming this agreement continues, could someone more knowledgable than me explain what would need to happen to rename an exercise please :) Do we need to PR to the different tracks that have implemented it? Is this automated/automatable? etc :) |
There hasn't been a rename in at least 2 years that I can recall, not as long as I have been around.... @kytrinyx should be able to tell us the best way to go about it.... |
So since exercises are now (presumably) identified in the database by their UUID, does it mean that tracks can freely and independently change their exercise slugs as they desire, without having to perform any database migrations, as we had to when exercises were identified in the database by their slug? If so, that's great! That means less manual work and no timing dependencies!
|
implementing tracks
|
This is correct, yes :) |
I'm really excited to hear that. Renaming a relatively new exercise will be a great way to trial out the process and see if there are any unexpected snags. Since there are only two implementing tracks the impact gets to be small. Then we'll be able to proceed with confidence in renaming other exercises. bracket-push has been on my list for a while #693. Since track slugs can be independently changed, that means as far as this repo is concerned, it can be treated as if a new exercise was created with the new name.
|
I have no objections to renaming retree. I'll let y'all figure out how to do it.
Let's not assume that every exercise will have a story. Unless having a "story" is documented as a requirement. And then you would also have to document exercise naming conventions. My point is that the documentation should be updated. I deffer to a more skilled writer to make the update (if having a story and using it for a name will actually be official). |
Python track maintainer here. I'm all for the rename. |
I think every exercise should have a story, and the documentation should be updated. But that's something I'll discuss with the product team and confirm before we make steps forward with :) |
Since this is the first time, I would be curious to hear a report of how it went. It will refine the process for the next time this comes up. At a surface glance, I can see https://exercism.io/my/tracks/python lists Satellite, so things look all right. https://exercism.io/my/tracks/prolog looks all right too as an example of what happens when a track has not yet performed. It shows Retree, so this is good as well. I have not attempted to fetch either exercise yet. |
I was able to fetch both exercises, so everything seems to be good from a student point of view. I have not checked the mentor point of view |
I just checked the mentor view for the Python track. Neither |
This commit renames the 'retree' exercise into 'satellite': - Decision: exercism/problem-specifications#1451 - PR: exercism/problem-specifications#1478
@Average-user: Can you confirm @cmccandless in this for the Prolog track? |
If it does not appear should be because is a side exercise and there is no uploaded solutions for it, right? |
Because exercises that have no submits but ARE core, appear in grey inside the list. But side exercises that have no submits do not. If someone submits a solution to that problem in the Prolog track I can check it out though |
I submitted a solution for We should be in the clear. |
I rehash my prior allegations of #693 below: I contend that having the word `push` in the name unnecessarily biases the solution space toward solutions that use a stack data structure (has a *push* operation) or a *push*-down automaton, rather than other solutions not using either of these two. We have heard a principle that we want to name exercises by their story, not by what they teach: #1451 (comment) The story here is about matching brackets, so I posit that that serves as the name we want. I have changed the major version number, as was done in the retree->satellite rename, even though I do not feel strongly about whether that should be necessary: #1478 As we have gained experience in the rename of retree, we see that problem-specifications is free to rename exercises at any point, without waiting for all tracks to follow suit. Of course, this operation should not be performed lightly since it causes churn in the 31 (*thirty-one*) tracks implementing this exercise.
At the end of last year a new exercise was added called "retree". I would like us to get into the habit of naming exercises by what the story is about, not what they teach. That makes the exercise more appealing in the UI (think
bob
vstrinary
orfood-chain
vstranspose
). I feel like this exercise is about transmitting data to Alpha Centauri - so I'd like to propose renaming it to "Satellite" instead. As it's a new exercise, I don't think this will be a big problem.What are people's thoughts? (cc @fidelcoria @sshine @ErikSchierboom @rpottsoh @nicolechalmers @kytrinyx)
The text was updated successfully, but these errors were encountered: