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

heads up: strain and accumulate are scheduled for deprecation #297

Closed
stkent opened this issue Feb 13, 2017 · 15 comments
Closed

heads up: strain and accumulate are scheduled for deprecation #297

stkent opened this issue Feb 13, 2017 · 15 comments
Labels

Comments

@stkent
Copy link
Contributor

stkent commented Feb 13, 2017

strain, accumulate

Wasn't sure of the best way to raise awareness of this otherwise. We should focus efforts on other exercises in lieu of improving these ones.

@jtigger
Copy link
Contributor

jtigger commented Feb 14, 2017

I see you're keeping an eye on x-common discussions! This is a really valuable thing to do! ;) Katrina is making a real go at delegating. There's a number of "Task Forces" that have kicked off and are clearly in full-stride in cleaning up the curriculum part of the platform.

@jtigger
Copy link
Contributor

jtigger commented Feb 14, 2017

So, the course of action here would be to move these exercises into the deprecated section in the config.json. Did you want to ... fold... that in to #286 ?

@stkent
Copy link
Contributor Author

stkent commented Feb 14, 2017

Since list-ops is not available in this track yet (though close!), I think it would be good to keep them around a while longer?

@stkent
Copy link
Contributor Author

stkent commented Feb 14, 2017

Nice pun, it was not unnoticed :D

@jtigger
Copy link
Contributor

jtigger commented Feb 14, 2017

Since list-ops is not available in this track yet (though close!)

painfully so! :) I'm so grateful to @javaeeeee's efforts there... it's really shaping up.

I think it would be good to keep them around a while longer?

Now, there you go making sense. :)

@jtigger
Copy link
Contributor

jtigger commented Feb 23, 2017

#207 has been merged! 🎉

@stkent are you happy with the way this all unfolded (i.e. notifying the track that these two exercises were deprecated)? Or do you think we can improve on this (e.g. literally adding in a @Deprecateds throughout the test suite/example)?

@stkent
Copy link
Contributor Author

stkent commented Feb 24, 2017

I think that leaking this information into the tests/examples would be overkill; however, deprecation being signaled by what seemed to be an somewhat-offhanded remark made by @kytrinyx in a generated issue in an upstream repository doesn't seem to be ideally visible! If there's a change to explore here, I would think it would be figuring out how to make that sort of decision more widely knowable. (There may well have been prior upstream discussion about those exercises that I missed, in which case, forgive my lack of knowledge!)

@kytrinyx
Copy link
Member

I think I have on my todo list somewhere to generate an issue for each track about deprecation of certain exercises, but everything's been a jumble for a while and I've lost track. I think those discussions happened in the x-common repository. I apologize for the confusion. I'd love it if we could lock down good procedures around all of this.

@stkent
Copy link
Contributor Author

stkent commented Feb 24, 2017

Generated issues would be perfect (from the POV of an actively-maintained track, at any rate)!

@kytrinyx
Copy link
Member

kytrinyx commented Feb 25, 2017

I found:

I'll update the discussions one to ask if someone wouldn't mind taking charge of figuring out exactly what has been decided, and what still needs to be done, and making it so.

Update: ok, the new issue that defines TODOs is linked to from the bottom of exercism/discussions#40

@stkent
Copy link
Contributor Author

stkent commented Feb 25, 2017

@kytrinyx ❤️ thanks for doing that digging and centralizing!

@stkent
Copy link
Contributor Author

stkent commented Feb 25, 2017

I notice that none of those issues mention strain being deprecated in favor of list-ops; is exercism/problem-specifications#588 (comment) the canonical reference for that decision? If so I'll add it to exercism/discussions#40 exercism/discussions#120 for reference.

@kytrinyx
Copy link
Member

Excellent find. I think that the comment you're referring to is the canonical reference (as much as we have one, anyway!)

@stkent
Copy link
Contributor Author

stkent commented Feb 25, 2017

Roger that; adding to exercism/discussions#40 exercism/discussions#120 as promised!

@stale
Copy link

stale bot commented May 6, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants