Skip to content

Encourage improvements after submission#262

Merged
kytrinyx merged 3 commits intoexercism:masterfrom
eToThePiIPower:submit_extra_hints
Dec 27, 2015
Merged

Encourage improvements after submission#262
kytrinyx merged 3 commits intoexercism:masterfrom
eToThePiIPower:submit_extra_hints

Conversation

@eToThePiIPower
Copy link
Contributor

As per issue #77, the displays extra hints after the users submit the first iteration of each problem.

The hints encourage users on ways to refactor, and then to participate in the community by commenting on their submission.

Variation is added through randomizing the opening phrase to keep users from automatically ignoring the message. Additional and more creative phrases here would be appreciated.

As per #77 I've added the "extra hints" from the "beer song" exercise
README to the output of the submit command. These hints provide
direction to help refactor, and then participate in the community
through commenting on the exercism site.

To keep annoyance down and keep users from getting used to (and thus
ignoring) them, the extra hints will show only on a successful
submission of the first iteration of each exercise.
To keep users from just automatically ignoring them, this adds some
variety to the extra hints after submitting the first iteration, by
randomly choosing from an assortment of header phrases.
cmd/submit.go Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Would you make sure this is run through gofmt, please? goimports, would be good, too.

Moving the long, static text block for the "extra tips" to a const
to reduce clutter in the Submit() function.
kytrinyx added a commit that referenced this pull request Dec 27, 2015
Encourage improvements after submission
@kytrinyx kytrinyx merged commit ca4110b into exercism:master Dec 27, 2015
@kytrinyx
Copy link
Member

Thanks!

In the future, after pushing a fix, would you please add a comment that it has been done? I don't get notifications when someone pushes to a PR.

@kytrinyx
Copy link
Member

kytrinyx commented Jan 9, 2016

Now that this has been in production for a bit... what do you think?

@Tonkpils
Copy link
Contributor

Tonkpils commented Jan 9, 2016

So, I've used it a bit since this was implemented and I feel that after
first or second submission it just becomes noise. It's good encouragement
initially, but I stopped reading it after my second submission.
On Sat, Jan 9, 2016 at 4:53 PM Katrina Owen notifications@github.com
wrote:

Now that this has been in production for a bit... what do you think?


Reply to this email directly or view it on GitHub
#262 (comment).

@kytrinyx
Copy link
Member

That's been my impression as well.

I think we should just take it back out, and keep working towards making the conversations on the site better.

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