Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change expected to expected_fh if it is a file handler #2864

Open
atharva-2001 opened this issue Oct 21, 2024 · 3 comments · May be fixed by #2949
Open

Change expected to expected_fh if it is a file handler #2864

atharva-2001 opened this issue Oct 21, 2024 · 3 comments · May be fixed by #2949
Assignees

Comments

@atharva-2001
Copy link
Member

expected_fh would clearly indicate that the variable holds a file handler. expected indicated data.

@Thakor-Yashpal
Copy link

"Can I work on this issue?"

@kevinzb56
Copy link

@atharva-2001 We can change expected to expected_fh in tardis.visualization.widgets.shell_info module.

@rudrakshnalbalwar
Copy link

Hi @atharva-2001 I wanted to confirm if the mentioned change needs to be applied across the entire codebase or only in specific sections/files. Could you please clarify? Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants