Skip to content

Adds context manager section to structure.rst#627

Merged
kennethreitz merged 1 commit into
realpython:masterfrom
sirMackk:master
Jan 15, 2016
Merged

Adds context manager section to structure.rst#627
kennethreitz merged 1 commit into
realpython:masterfrom
sirMackk:master

Conversation

@sirMackk

Copy link
Copy Markdown
Contributor

I've noticed that there was no mention of context managers in the guide. Personally I've found them to be a way underrated mechanism to manage complexity even in small projects.

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.

Would you mind adding "at runtime" after "to an action"/

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's not necessarily running functions (as you show with the second example you provided).

@sigmavirus24

Copy link
Copy Markdown
Contributor

Hey @sirMackk ! Thanks for putting this together. It's a really great start. I have some changes I'd love for you to make and a question in there about one of your examples.

Ping me here when you've made changes you'd like me to review.

Cheers,
Ian

kennethreitz added a commit that referenced this pull request Jan 15, 2016
Adds context manager section to structure.rst
@kennethreitz kennethreitz merged commit 6006150 into realpython:master Jan 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants