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.

Sign up for GitHub

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

Jump to bottom

Rework ShellLogger into a context manager #35

Open
jmgate opened this issue Apr 25, 2024 · 0 comments
Open

Rework ShellLogger into a context manager #35

jmgate opened this issue Apr 25, 2024 · 0 comments
Labels
Type: Refactor Refactoring existing code to improve long-term sustainability.

Comments

@jmgate
Copy link
Collaborator

jmgate commented Apr 25, 2024

INSERT DETAILS HERE.

The user shouldn't have to call finalize()—it should happen automatically.

  • What needs to be cleaned up?
  • How would the refactored implementation differ from what we have now?
  • What are the potential downsides to not dealing with the technical debt now and leaving it for later?
@jmgate jmgate added the Type: Refactor Refactoring existing code to improve long-term sustainability. label Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: Refactor Refactoring existing code to improve long-term sustainability.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jmgate