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

Overhaul the track README #37

Merged
merged 1 commit into from
Jul 28, 2017
Merged

Overhaul the track README #37

merged 1 commit into from
Jul 28, 2017

Conversation

kytrinyx
Copy link
Member

I've tried to deduplicate as much as possible, pointing to shared Exercism documentation where we have it, and only referring to language-specific changes here. I've left in a bit of high-level stuff for the sake of readability/flow.

Fixes #35
Fixes #36

I've tried to deduplicate as much as possible, pointing to shared Exercism documentation where we have it, and only referring to language-specific changes here. I've left in a bit of high-level stuff for the sake of readability/flow.
@kytrinyx kytrinyx requested a review from petertseng July 28, 2017 19:57
- Please do not commit any configuration files or directories inside the exercise other than ...

- Be sure to add it to the appropriate place in the `config.json` file. Also, please run `bin/fetch-configlet && bin/configlet` to ensure the exercise is configured correctly.
Copy link
Member

Choose a reason for hiding this comment

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

good move, because otherwise the two things done by Travis CI are split up in a weird place


- Exercises should use only the {{LANGUAGE}} core libraries.
Please see the documentation about [adding new exercises](https://github.com/exercism/docs/blob/master/you-can-help/make-up-new-exercises.md).
Copy link
Member

Choose a reason for hiding this comment

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

will there also be a https://github.com/exercism/docs/blob/master/you-can-help/implement-an-exercise-from-specification.md section? Does the distinction no longer exist, perhaps (and we only care about in the track vs not in the track)?

Copy link
Member Author

@kytrinyx kytrinyx Jul 28, 2017

Choose a reason for hiding this comment

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

Oh, right. Hm.

I think that we should probably have separate documentation for:

  • generic problem specification
  • custom problem specification
  • details about implementing an exercise

And then have two guides that refer to the more fleshed out data as "use cases". E.g. You want to add a brand new exercise? Here's what you need to think about. You want to implement an exercise based on a generic specification? Here's what to think about.

Copy link
Member Author

Choose a reason for hiding this comment

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

I think we have some of this in https://github.com/exercism/docs/tree/master/language-tracks/exercises, so we could clean that up a bit.

I'm trying to get the documentation structured so that most of the conceptual / reference stuff is in the right place, and then the guides (do a particular task) refers to those guides. I'm still not quite there yet.

Copy link
Member Author

Choose a reason for hiding this comment

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

OK, I've thought about this for a bit. I think my solution is going to be to rename the "port an exercise" guide to be "implement an exercise" (implied: given a problem specification, either general or custom).

I'll do this in a separate set of PRs, since I'll need to rename references to it across all of Exercism.

@kytrinyx kytrinyx merged commit 96d7f70 into master Jul 28, 2017
@kytrinyx kytrinyx deleted the track-readme branch July 28, 2017 22:21
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