Skip to content

Conversation

aharrison24
Copy link
Contributor

This is a suggested solution to #14 for the exercise in chapter 3. I think the main thing is just to let readers know that they will be given a solution to the exercise later, so they don't need to be worried if they're unable to solve it.

I also added a new line before the Exercise, so that it appeared on its own line rather than being inline with the previous paragraph.

I did try to put in a relative link to the chapter, but it wasn't clear what the best practice for that was. Something like [chapter 6](./06_min_max.html) works, but hard coding the .html suffix seems a bit icky.

@justinmeiners
Copy link
Owner

justinmeiners commented Sep 14, 2021

Style options:

  1. Footnote, as illustrated in this PR. "The answer to this exercise is found in chapter X."
  2. Parenthetical. (solved in chapter X).
  3. Additional sentence at the end of the exercise statement: "The answer is found in chapter X."

I think I like 2 or 3 because in a few exercises I provide the solution in a footnote. So, we could tell the reader the solution is in the footnote to give them a chance to solve it before seeing the answer.

It would be really nice if it could link to the other chapter in a way that works in markdown and html, so that we don't have to remember the chapter numbers, but I don't think there is an easy way to do this in markdown.

Do you have any thoughts?
I would also like @rpendleton opinion.

@aharrison24
Copy link
Contributor Author

Any of the three works for me, with a very slight preference for 2. I agree that 1 is less desirable when some exercises are actually solved in a footnote.

@justinmeiners
Copy link
Owner

Great, let's do 2.

@justinmeiners
Copy link
Owner

I have added a file .github/STYLE.md to track these decisions.

@aharrison24
Copy link
Contributor Author

Nice! I've updated this PR to follow the shiny new style guide.

@justinmeiners justinmeiners merged commit ed291ba into justinmeiners:master Sep 19, 2021
@aharrison24 aharrison24 deleted the identity-violation branch September 19, 2021 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants