Skip to content

Fix the way the output of table is printed in print()#12895

Merged
devyn merged 1 commit intonushell:mainfrom
devyn:fix-table-output-bytestream-printing
May 17, 2024
Merged

Fix the way the output of table is printed in print()#12895
devyn merged 1 commit intonushell:mainfrom
devyn:fix-table-output-bytestream-printing

Conversation

@devyn
Copy link
Copy Markdown
Contributor

@devyn devyn commented May 17, 2024

Description

Forgot that I fixed this already on my branch, but when printing without a display output hook, the implicit call to table gets its output mangled with newlines (since #12774). This happens when running nu -c or a script file.

Here's that fix in one PR so it can be merged easily.

Tests + Formatting

  • 🟢 toolkit fmt
  • 🟢 toolkit clippy
  • 🟢 toolkit test
  • 🟢 toolkit test stdlib

@devyn devyn marked this pull request as ready for review May 17, 2024 13:56
@devyn devyn merged commit 59f7c52 into nushell:main May 17, 2024
@devyn devyn deleted the fix-table-output-bytestream-printing branch May 17, 2024 14:18
@hustcer hustcer added this to the v0.94.0 milestone May 17, 2024
FilipAndersson245 pushed a commit to FilipAndersson245/nushell that referenced this pull request May 18, 2024
)

# Description

Forgot that I fixed this already on my branch, but when printing without
a display output hook, the implicit call to `table` gets its output
mangled with newlines (since nushell#12774). This happens when running `nu -c`
or a script file.

Here's that fix in one PR so it can be merged easily.

# Tests + Formatting
- 🟢 `toolkit fmt`
- 🟢 `toolkit clippy`
- 🟢 `toolkit test`
- 🟢 `toolkit test stdlib`
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