Skip to content

Code blocks#393

Merged
sigmavirus24 merged 13 commits into
realpython:masterfrom
Zearin:code-blocks
Apr 2, 2014
Merged

Code blocks#393
sigmavirus24 merged 13 commits into
realpython:masterfrom
Zearin:code-blocks

Conversation

@Zearin
Copy link
Copy Markdown
Contributor

@Zearin Zearin commented Mar 28, 2014

Hey, @kennethreitz! Love your work, and this book. :)

I made some “janitorial” type edits here. They are almost entirely related to specifying the syntax for code-blocks.

I figured I’d start simple, because I’m not used to writing in ReST. That said…I honestly have no idea if I broke anything. (ReST is powerful, but I can never get the syntax straight from memory, so I usually prefer markdown).

If I did break something, just comment on the broken part(s) in the diff, and I’ll fix it up.

If this works out, I have some more edits in the works…

@sigmavirus24
Copy link
Copy Markdown
Contributor

Hey @Zearin thanks for this! There is going to be some work necessary before this is merged though. You made some unnecessary changes in certain places, but on the whole this is fantastic. Thank you! 🍰

Comment thread Readme.rst Outdated
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.

Might as well split these up. They are separate tools although they are very related. They should also both be capitalized, e.g., Pip.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed in ad0bba3

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

RE-fixed in 5f86330 (forgot the capitalization! :P)

@sigmavirus24
Copy link
Copy Markdown
Contributor

Also, for each thing you fix, reply to each of my comments with "Fixed in ". If possible, keep the commits as small as possible.

@Zearin
Copy link
Copy Markdown
Contributor Author

Zearin commented Mar 31, 2014

Hey @Zearin thanks for this! There is going to be some work necessary before this is merged though. You made some unnecessary changes in certain places, but on the whole this is fantastic. Thank you!

You’re welcome! :)

Gonna start on your comments shortly…

@Zearin
Copy link
Copy Markdown
Contributor Author

Zearin commented Mar 31, 2014

Okay @sigmavirus24…fixed the last of your comments! Let me know if there is anything else I should fix. :)

P.S.: (BTW, is your handle a Mega Man X reference? God that was an awesome game…)

@sigmavirus24
Copy link
Copy Markdown
Contributor

It is a reference to that. I think you're the first to get it. An extra slice of 🍰 for you.

Thanks for all of this!

sigmavirus24 added a commit that referenced this pull request Apr 2, 2014
@sigmavirus24 sigmavirus24 merged commit 9d8a491 into realpython:master Apr 2, 2014
@Zearin Zearin deleted the code-blocks branch April 4, 2014 20:28
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