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

Improves the output of "yarn config" #5713

Merged
merged 16 commits into from
Sep 28, 2023
Merged

Improves the output of "yarn config" #5713

merged 16 commits into from
Sep 28, 2023

Conversation

arcanis
Copy link
Member

@arcanis arcanis commented Sep 1, 2023

What's the problem this PR addresses?

It's a little difficult to ask people to check specific values of their configuration at the moment. They need to run yarn config get <name> to see the value, or yarn config --why | grep <name>, but that isn't very visible.

How did you fix it?

It's a bit of an experiment to try to change the output from:

image

To the new tree display:

image

To offset the lack of compaction, the command now also accepts a list of setting names on the command line, so you can write the following:

yarn config nodeLinker

Or even multiple parameters:

yarn config supportedArchitectures enableGlobalCache
image

Checklist

  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@arcanis arcanis force-pushed the mael/config-display branch from 637a6d9 to fa9d164 Compare September 26, 2023 10:07
@arcanis arcanis merged commit 48327b2 into master Sep 28, 2023
@arcanis arcanis deleted the mael/config-display branch September 28, 2023 13:49
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.

1 participant