Skip to content

Fixed #82 -- Improved the chapter on dictionaries.#107

Merged
olasitarska merged 1 commit intoDjangoGirls:masterfrom
bmispelon:dict-improvement
Aug 10, 2014
Merged

Fixed #82 -- Improved the chapter on dictionaries.#107
olasitarska merged 1 commit intoDjangoGirls:masterfrom
bmispelon:dict-improvement

Conversation

@bmispelon
Copy link
Copy Markdown
Contributor

Thanks to @kerstin for the feedback.

Comment thread python_introduction/README.md Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should be: an empty dictionary

@olasitarska
Copy link
Copy Markdown
Member

I added some feedback here. Do you mind fixing?:)

@bmispelon
Copy link
Copy Markdown
Contributor Author

Of course not, I'll fix it as soon as I'm back at the computer.

Thanks!
On Aug 9, 2014 12:44 PM, "Ola Sitarska" notifications@github.com wrote:

I added some feedback here. Do you mind fixing?:)


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

@bmispelon
Copy link
Copy Markdown
Contributor Author

@olasitarska: I fixed all the mistakes you found, thanks.

@oinopion @kerstin: Does that look good? Is there something else I could address in this PR?

@olasitarska
Copy link
Copy Markdown
Member

@bmispelon thanks! ready to be merged for me, but let's hear from @oinopion and @kerstin :)

@keikoro
Copy link
Copy Markdown

keikoro commented Aug 10, 2014

Looking goooooooooood! Thanks so much for the improvement, @bmispelon. ((:

olasitarska added a commit that referenced this pull request Aug 10, 2014
Fixed #82 -- Improved the chapter on dictionaries.
@olasitarska olasitarska merged commit 98e4423 into DjangoGirls:master Aug 10, 2014
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