Skip to content

two-fer: add to track#663

Merged
stkent merged 7 commits into
exercism:masterfrom
Smarticles101:two-fer
Jun 26, 2017
Merged

two-fer: add to track#663
stkent merged 7 commits into
exercism:masterfrom
Smarticles101:two-fer

Conversation

@Smarticles101

Copy link
Copy Markdown
Member

Reviewer Resources:

Track Policies

@Smarticles101 Smarticles101 changed the title two-fer: dibs on implementing two-fer: add to track Jun 23, 2017
@stkent stkent self-assigned this Jun 24, 2017
@Smarticles101

Copy link
Copy Markdown
Member Author

I was thinking since this is meant to be right after hello world, should we add any readme files that the user might need to read? The description of this exercise introduces test driven development - would it be appropriate to remove @Ignores from the hello-world exercise and introduce them here instead?

@stkent

stkent commented Jun 24, 2017

Copy link
Copy Markdown
Contributor

@Smarticles101 you are right that the purpose of adding this exercise was to allow Hello, World to become as simple as possible, thereby testing user environment, etc. Removing Ignores from Hello, World, and introducing them in this exercise instead supports that goal, so I say 👍

@Smarticles101

Copy link
Copy Markdown
Member Author

How does this look @stkent?

Comment thread config.json
},
{
"slug": "two-fer",
"difficulty": 1,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This topics entry should be formatting like the others (with a blank line between the brackets).

@stkent

stkent commented Jun 24, 2017

Copy link
Copy Markdown
Contributor

I think this update to the documentation is good for now, though looking at the content of the two exercises it is clear that we should separately split that content between the two exercises.

Ref.:

@stkent

stkent commented Jun 24, 2017

Copy link
Copy Markdown
Contributor

I'm happy to take on the separate work I mention after this PR is merged.

@stkent

stkent commented Jun 24, 2017

Copy link
Copy Markdown
Contributor

@Smarticles101 just one change to make in the config formatting, then LGTM!

@Smarticles101

Copy link
Copy Markdown
Member Author

@stkent ah, so two-fer is a replacement for what hello-world was, and hello-world should just become the traditional hello world.

@stkent

stkent commented Jun 24, 2017

Copy link
Copy Markdown
Contributor

Yep, exactly :)

@stkent

stkent commented Jun 24, 2017

Copy link
Copy Markdown
Contributor

Ok, this is ready to merge, I'm just going to wait on hitting the button until I have 30 mins right after to update the documentation around this and hello-word :)

@stkent

stkent commented Jun 26, 2017

Copy link
Copy Markdown
Contributor

I'll be taking care of this later this afternoon.

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