Skip to content

Update 7.1-exception.ipynb#7

Open
kumarmohit0911 wants to merge 1 commit into
krishnaik06:mainfrom
kumarmohit0911:my_PR
Open

Update 7.1-exception.ipynb#7
kumarmohit0911 wants to merge 1 commit into
krishnaik06:mainfrom
kumarmohit0911:my_PR

Conversation

@kumarmohit0911

Copy link
Copy Markdown

In the very third cell from bottom u typed <file.closed()> but <file.close> here there won't be any parenthesis as closed is not a method but a property which returns boolean value. And its not throwing error bcoz it is not getting executed as u have used operator.

In the very third cell from bottom u typed <file.closed()> but <file.close>
here there won't be any parenthesis as closed is not a method but a property which returns boolean value. And its not throwing error bcoz it is not getting executed as u have used or operator.

@mjbear mjbear left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I agree that file.closed is a Boolean property and not a method.

@mjbear

mjbear commented Nov 18, 2024

Copy link
Copy Markdown

Now it's up to @krishnaik06 for the final review and decision whether to merge these changes!

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