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

consider using environment vars for terminal #77

Closed
wants to merge 1 commit into from

Conversation

bdmorin
Copy link

@bdmorin bdmorin commented May 11, 2022

Adding environment variables for columns and lines can fix terminal issues when entering a container.
also
-e COLUMNS=$COLUMNS -e LINES=$LINES -e TERM=$TERM if no command subprocess is desired

Discord username (optional) please include so we can attribute you with our Contributor role (like so elvis#4747)

Description of changes (updated or new workflows)

Adding environment variables for columns and lines can fix terminal issues when entering a container.
also 
```-e COLUMNS=$COLUMNS -e LINES=$LINES -e TERM=$TERM``` if no command subprocess is desired
@alokedesai
Copy link
Member

Is this still an open issue with Docker? My understanding is this was fixed a few years ago, see moby/moby#33910 and moby/moby#37172

@alokedesai
Copy link
Member

@bdmorin Friendly ping on this! Mind addressing the comment or closing out if this is no longer relevant. Thanks!

@alokedesai
Copy link
Member

Closing this out per my comment above. Feel free to reopen if you're still working on this!

@alokedesai alokedesai closed this Jul 22, 2022
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.

None yet

2 participants