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

Add documentation about contributing to tracks #3232

Merged
merged 9 commits into from
Nov 12, 2016
Merged

Conversation

kytrinyx
Copy link
Member

@kytrinyx kytrinyx commented Nov 4, 2016

@exercism/track-maintainers I've been working on documenting more of the work that's involved in contributing to a language track. I think that if we can get this properly documented, we can do a bit more around explicitly trying to find more people to contribute (and eventually to maintain) the tracks.

I'd love feedback about anything that catches your eye. Clarity, missing topics, correctness, whatever it is.

View the rendered files:

* How do you run the tests? Is there a way to run all the tests for all the exercises?
* Are there any naming conventions for files or types or classes or functions?
* Is there any tooling that we're using? Linters?
* Is there continuous integration? Are there any gotchas?
Copy link

Choose a reason for hiding this comment

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

Hints or links to resources for the language? (e.g. man test for learning conditionals in bash)

Copy link
Member Author

Choose a reason for hiding this comment

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

Oooh, yes. I think that could be part of improving the exercises, same as reordering. I think maybe we should have an "As you solve exercises..." section. Things that you can think about that might cause you to improve the exercise.

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 reorganized the page a little bit, and added a section about adding hints.

### Porting an Exercise

The easiest way to add a new exercise is to find an exercise that has already been implemented in another language
track, and port it over to your target language.
Copy link

Choose a reason for hiding this comment

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

I would be curious to see a list of the available exercises. Is https://github.com/exercism/x-common/blob/master/exercises/ the best place to start?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, that's the complete list of exercises. It doesn't have any information about what's easy or hard, though.

We also have:

The UX on that last one is kind of awful, but it works. The linked documentation has more of a step-by-step thing.

Copy link

@bkeepers bkeepers left a comment

Choose a reason for hiding this comment

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

This is great! I haven't gotten through all the docs, but a few comments to start.

@ErikSchierboom
Copy link
Member

Great initiative! I've submitted some grammar improvement PR's. Looks great to me.

Copy link

@parkerl parkerl left a comment

Choose a reason for hiding this comment

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

These are great. Couple small things.


The goal of triaging issues is to make sure that each issue is actionable.

I great issue also has enough context that it could be tackled by someone who has never contributed
Copy link

Choose a reason for hiding this comment

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

"A great issue..."?

Copy link
Member Author

Choose a reason for hiding this comment

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

Hah. Yeah, that. Thanks :)

# Triaging Issues in an Exercism Language Track

Issues need to be unambiguous and actionable. If it's hard to figure out what
an issue is about, or what to do about it, then the issue probably won't get
Copy link

Choose a reason for hiding this comment

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

"...out what an issue is about, or what needs to be done,"

@kytrinyx
Copy link
Member Author

I'm going to go ahead and merge this. I always appreciate pull requests and further comments, so don't hesitate to have opinions later, too.

@kytrinyx kytrinyx merged commit 84aba8a into master Nov 12, 2016
@kotp kotp deleted the language-track-docs branch February 19, 2017 22:55
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.

None yet

5 participants