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

change: modified description.md to match expected output #1339

Merged
merged 1 commit into from
Sep 24, 2018

Conversation

sdublish
Copy link
Contributor

Closes #1283

This makes it so that description.md matches the output expected based off canonical-data.json.

Copy link
Member

@petertseng petertseng left a comment

Choose a reason for hiding this comment

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

I agree that this makes README match canonical-data. I will not today be expressing an opinion on which of the two possible directions the sync should go in, I only say that this is the correct execution of this direction.

I used to be able to see what tracks had implemented an exercise, but this is more difficult now. I might have liked to examine how tracks are doing it, but without the information it is inconvenient for me to do so (though still possible)

@petertseng
Copy link
Member

Yes, for better or worse, the tracks that had this exercise in July (the most recent data available) all use the full list of coins rather than the list of counts. Those tracks are

clojure/exercises/change
csharp/exercises/change
ecmascript/exercises/change
elixir/exercises/change
fsharp/exercises/change
go/exercises/change
haskell/exercises/change
java/exercises/change
javascript/exercises/change
kotlin/exercises/change
lua/exercises/change
ocaml/exercises/change
php/exercises/change
python/exercises/change
reasonml/exercises/change
ruby/exercises/change
scala/exercises/change

@petertseng petertseng merged commit e926a23 into exercism:master Sep 24, 2018
@petertseng
Copy link
Member

Thanks!

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