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

options.format: lines (output packages line by line) #1283

Merged

Conversation

vanodevium
Copy link
Contributor

#1282

@raineorshine I have some problems with TS :(

Anyway, I'm trying.

Is this correct implementation? Can I continue work with tests and readme?

@raineorshine
Copy link
Owner

raineorshine commented Mar 21, 2023

This looks like the right approach! You can run npm run build to re-generate the README based on the cli-options.

Let's just have --format lines. That is the most unixy, and can easily be converted into other formats in the shell. npm install -g $(ncu -g --format lines) works as expected.

e.g.

ncu-test-v2@2.0.0
ncu-test-tag@1.1.0

src/lib/initOptions.ts Outdated Show resolved Hide resolved
@vanodevium
Copy link
Contributor Author

@raineorshine Maybe --format unix? lines is ambiguous as for me.

@raineorshine
Copy link
Owner

--format unix is ambiguous in its own way, and I don't want to imply that this feature is platform specific and can't be used by Windows users. --format lines describes the output: one update per line, without anything else. The help system can clarify further.

@vanodevium
Copy link
Contributor Author

OK. I have done my best.

@vanodevium vanodevium changed the title WIP Options format: shell, lines, comma options.format: lines (output packages line by line) Mar 22, 2023
test/format.test.ts Show resolved Hide resolved
@raineorshine raineorshine merged commit 4d825c5 into raineorshine:main Mar 22, 2023
@vanodevium vanodevium deleted the options-format-shell-lines-comma branch March 22, 2023 20:22
@vanodevium
Copy link
Contributor Author

Thanx!

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