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

suggestion: remove trailing white space #46

Open
tiagolr opened this issue Apr 15, 2017 · 0 comments · May be fixed by #69
Open

suggestion: remove trailing white space #46

tiagolr opened this issue Apr 15, 2017 · 0 comments · May be fixed by #69

Comments

@tiagolr
Copy link

tiagolr commented Apr 15, 2017 •
edited
Loading

I'm reducing the columnified outputs by as much as half by applying this operation on the results:

columnify(data).replace(/\s+$/gm, '')

this matters when sending columnified logs over the web, or storing them, leave you with this suggestion
abitrolly added a commit to abitrolly/columnify that referenced this issue Sep 15, 2023
@abitrolly
Strip trailing whitespace from last column
76539f1 
Fixes timoxley#46
@abitrolly abitrolly linked a pull request Sep 15, 2023 that will close this issue
Open
chenrui333 pushed a commit to abitrolly/homebrew-core that referenced this issue Sep 16, 2023
@abitrolly @chenrui333
web-ext 7.7.0 (new formula)
ee566f2 
web-ext: fix tap_syntax failures

web-ext: assert command response in test

`web-ext lint` output is made by `addons-linter` and it contains
trailing whitespace from `columnify` library

https://github.com/mozilla/addons-linter/blob/025e033696b42b27753ae31cbf8d5dde8138d301/src/linter.js#L174-L180

Related `columnify` issue

timoxley/columnify#46

web-ext: tap_syntax fixes

web-ext: copy terminal-notifier fix from vue-cli

web-ext: update test

Signed-off-by: Rui Chen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Strip trailing whitespace from last column abitrolly/columnify
1 participant
@tiagolr