Skip to content

parallel-letter-frequency: Mark exercise as foregone #1107

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 3 commits into from
Nov 16, 2017

Conversation

N-Parsons
Copy link
Contributor

Resolves #1106 by marking parallel-letter-frequency as foregone and removing the exercise directory.

I'm open to discussion on whether the directory and (incorrect) implementation should remain in the repo, but I have removed it because the current implementation is wrong (and not parallel) and so I feel that it holds no value to retain it.

@N-Parsons
Copy link
Contributor Author

I'm going to go ahead and merge this now, since there are issues with this exercise that could put learners off.

@N-Parsons N-Parsons merged commit a7a4379 into exercism:master Nov 16, 2017
@cg-cnu
Copy link

cg-cnu commented Nov 16, 2017

Thanks @N-Parsons for removing it. Just working on it and the title and instructions are not clear and seems to be a bottleneck for many people to move forward.
I guess the numbers says it all

image

@N-Parsons
Copy link
Contributor Author

@cg-cnu, it was a fairly recent addition to the track (9 days ago), which is most of the reason why the numbers are so low. Hopefully people weren't too put off during that time - we'll be more careful in future with where in the config each exercise is being placed, although the upcoming revision to the way Exercism works should also stop this kind of thing from happening.

@cg-cnu
Copy link

cg-cnu commented Nov 16, 2017

Hey @N-Parsons Sorry, its a misunderstanding. Didn't know that its a recent addition. You guys are doing awesome work ❤️ Thanks for your time 🙇

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