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

Provide more info in package summary #455

Closed
aronovgj opened this issue Oct 8, 2015 · 9 comments
Closed

Provide more info in package summary #455

aronovgj opened this issue Oct 8, 2015 · 9 comments
Assignees
Milestone

Comments

@aronovgj
Copy link

aronovgj commented Oct 8, 2015

After cup all -pre --noop we get something like this:

Chocolatey can upgrade 1/139 package(s).
 8 package(s) had warnings.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).
Warnings:
 - praatcon
 - chromium
 - hitmanpro
 - pidgin-otr
 - 4t-tray-minimizer
 - powershell
 - github
 - keyboard-layout-creator

I'd like a little more info. Something like this:

Chocolatey can upgrade 1/139 package(s).
- javaruntime | 8.0.40 -> 8.0.51
 8 package(s) had warnings.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).
Warnings:
 - praatcon | not in source list
 - chromium | pinned; skipped
 - hitmanpro | not in source list
 - pidgin-otr | not in source list
 - 4t-tray-minimizer | not in source list
 - powershell | pinned; skipped
 - github | pinned; skipped
 - keyboard-layout-creator | not in source list

Else I find the summary to be pretty useless.

Probably another issue but there should be a distinction between not actually being in the source list and being in the source list while not yet being moderated the first time.

@RichiCoder1
Copy link
Contributor

So an improved version of choco upgrade all --noop -verbose?

@aronovgj
Copy link
Author

aronovgj commented Oct 8, 2015

--verbose does not have any effect on cup as far as I see it (don't have any packages to upgrade at the moment so they might return more info when they're upgraded). I'm concerned with the summary at the end of the output which should be more informative. I actually don't care so much that a hundred packages are the latest version available based on your source(s).. I'm interested in which of them are the exception to what should be the state of every package (being up to date).

@ferventcoder ferventcoder added this to the 0.9.10.x milestone Oct 10, 2015
@ferventcoder ferventcoder changed the title [Enhancement] More info in summary after upgrades More info in summary after upgrades Oct 10, 2015
@ferventcoder ferventcoder modified the milestones: 0.9.10, 0.9.10.x May 30, 2016
@ferventcoder ferventcoder self-assigned this May 30, 2016
@ferventcoder
Copy link
Member

This was completed as part of #735.

@ferventcoder ferventcoder changed the title More info in summary after upgrades Provide more info in package summary May 30, 2016
ferventcoder added a commit that referenced this issue May 30, 2016
When providing the summary of an install/upgrade/uninstall, there is a
list of packages that had warnings and errors. If there is a warning or
error message to go along with that, provide that as part of the
summary.
ferventcoder added a commit that referenced this issue May 30, 2016
* stable:
  (doc) update CHANGELOG/nuspec
  (maint) formatting
  (GH-735) Remove actual when uninstalling meta
  (GH-455) Summary - log reason for warnings/errors
  (GH-735) split package list from all operation
  (GH-710) If timeout, inform user
  (maint) add review note
  (GH-689) Report install location
  (GH-755) Track more MSI information
  (maint) Template specs - restrict to Windows only
@aronovgj
Copy link
Author

No summary showing which packages were upgraded? I still only see warnings (and I like how they look now).

@ferventcoder
Copy link
Member

@aronovgj I'm not sure what you mean.

Are you saying you want a summary of the packages that were upgraded?

@aronovgj
Copy link
Author

Yes

@aronovgj
Copy link
Author

aronovgj commented May 31, 2016

As in

Chocolatey has upgraded 1/139 package(s).
- javaruntime | 8.0.40 -> 8.0.51

@ferventcoder
Copy link
Member

I didn't see that in the issue context - however it does sound like a pretty good idea - please file a new ticket. Thanks!

@ferventcoder
Copy link
Member

clarification - I see it now (I didn't see it originally).

ferventcoder added a commit that referenced this issue Jun 11, 2016
Update the summary when warning about unfound sources so that only the
first sentence is shown in the package warning summary. This reduces
the amount of text seen in the summary.
ferventcoder added a commit that referenced this issue Jun 11, 2016
* stable:
  (GH-715) Report loaded extension in debug
  (GH-455) Adjust summary for unfound sources
  (GH-557) Adjust logging of upgrade messages
  (GH-759) Summarize affected pkgs when at least 5
  (GH-759) One action summary for all actions
  (maint) rename method
  (GH-742) Feature List Formatting
  (maint) licensing updates
  (doc) update generated docs / add download
  (maint) Allow better use of PowershellService
  (GH-781) Web Functions - Return early when local file
  (doc) add more information regarding pro/business
  (GH-563) Environment changed message as warning
  (doc) Uninstall-ChocolateyZipPackage is optional
  (doc) Point to new docs location
  (maint) do not log warning unless regular output
  (GH-519) Update Install-ChocolateyShortcut Docs
  (doc) update generated docs
  (GH-773) Autodoc - Examples Before Parameters
  (GH-774) Autodoc - Fix Aliases
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

4 participants