Skip to content

Editorial tweaks + how to print a given element of a dictionary#102

Merged
bmispelon merged 2 commits intoDjangoGirls:masterfrom
MarysiaLowas:master
Aug 8, 2014
Merged

Editorial tweaks + how to print a given element of a dictionary#102
bmispelon merged 2 commits intoDjangoGirls:masterfrom
MarysiaLowas:master

Conversation

@MarysiaLowas
Copy link
Copy Markdown
Contributor

Just a bunch of editorial tweaks really. I've also added an example how to print a given element of a dictionary.

Comment thread django_orm/README.md 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.

It should be "how will we achieve it" here. You mind fixing that too?

@bmispelon
Copy link
Copy Markdown
Contributor

This is some seriously awesome feedback, thanks for taking the time to fix this.

I've left a few minor comments. If you can fix those, I'll be happy to press the big shiny green "merge" button :)

Thanks again!

@bmispelon
Copy link
Copy Markdown
Contributor

To include the fix as part of the pull request, just make the changes on your computer then make a commit and push it back to github.

This should automatically update the pull request and even hide my comments (github is smart enough to detect that my comments are outdated).

bmispelon added a commit that referenced this pull request Aug 8, 2014
Editorial tweaks + how to print a given element of a dictionary
@bmispelon bmispelon merged commit 1266c23 into DjangoGirls:master Aug 8, 2014
@bmispelon
Copy link
Copy Markdown
Contributor

Dziękuję bardzo!

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