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

Enable install logging & jsonUpgraded doesn't log #1334

Open
3 tasks done
HamoBoker opened this issue Sep 17, 2023 · 3 comments
Open
3 tasks done

Enable install logging & jsonUpgraded doesn't log #1334

HamoBoker opened this issue Sep 17, 2023 · 3 comments

Comments

@HamoBoker
Copy link

HamoBoker commented Sep 17, 2023

  • I have searched for similar issues
  • I am using the latest version of npm-check-updates
  • I am using node >= 14.14

Hi, i want to ask if there is a way to log package-manager logs or progress bar after running install script?

also i think i'm facing an issue with jsonUpgraded it doesn't log updated dependencies after process complete (it was logging in earlier version) so i'm not sure if its a bug or i'm missing something.

Thanks for this amazing work


.ncurc:

{
  "cache": true,
  "upgrade": true,
  "interactive": true,
  "jsonUpgraded": true,
  "color": true,
  "concurrency": 8,
  "packageManager": "pnpm",
  "format": ["group", "ownerChanged", "time", "lines"]
}

Current Behavior

jsonUpgraded Doesn't log updated dependencies after complete (it was logging in earlier version)

process of ncu command with the specified config above

Screen Recording 2023-09-17 at 1 26 24 PM

Expected Behavior

Output updated dependencies

@raineorshine
Copy link
Owner

raineorshine commented Sep 17, 2023

Hi, i want to ask if there is a way to log package-manager logs or progress bar after running install script?

Yes, that's a good idea. I have added this as the default behavior in v16.14.4.

also i think i'm facing an issue with jsonUpgraded it doesn't log updated dependencies after process complete (it was logging in earlier version) so i'm not sure if its a bug or i'm missing something.

The format: lines options is rather special: It cannot be combined with jsonUpgraded or other format options. So if you pick one or the other it should work fine. I will add better error messaging for conflicting options.

For future reference, please create separate issues for separate points of inquiry, regardless of how small they are. That will help me track conversations and progress independently.

@HamoBoker
Copy link
Author

thank you wo much, and my apologies next time I'll be more organized 😅

@raineorshine
Copy link
Owner

No worries. Thanks for bringing these to my attention. npm-check-updates is better because of your feedback :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants