search

binref / refinery

High Octane Triage Analysis
Other
618 stars 62 forks source link

Fixed docstring reference to REFINERY_TERM_SIZE environment variable #31

Closed Sevaarcen closed 1 year ago

Sevaarcen commented 1 year ago

While trying to use binref in a non-interactive terminal session, I ran into issues with the terminal columns being 0 and causing errors.I saw the REFINERY_TERMSIZE environment variable referenced in the get_terminal_size method which would fix my issue, however it didn't actually work.

After looking at the code more in depth, I noticed this is because the correct environment variable is REFINERY_TERM_SIZE, which is parsed out here https://github.com/binref/refinery/blob/67b9bdfc79ab85cbaacf59b3df5f15a23ef46a11/refinery/lib/environment.py#L160

This PR is to fix the docstring of the get_terminal_size method to reference the correct environment variable spelling.

huettenhain commented 1 year ago

Whoops, great catch. Thank you for the fix!