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

Include package name in verbose report #2349

Merged
merged 1 commit into from
May 23, 2017
Merged

Include package name in verbose report #2349

merged 1 commit into from
May 23, 2017

Conversation

agross
Copy link
Contributor

@agross agross commented May 18, 2017

Instead of reporting unprocessed dependencies like this:

   Retrieved Explored Package  FluentAssertions 4.19.0
   1 packages in resolution.
     - FluentAssertions, 4.19.0
   4 requirements left
     - D:\dev\heco\comwork\tools\paket.dependencies, >= 3.3 < 4.0
     - D:\dev\heco\comwork\tools\paket.dependencies, >= 3.0 < 4.0
     - D:\dev\heco\comwork\tools\paket.dependencies, >= 2.0 < 3.0
     - D:\dev\heco\comwork\tools\paket.dependencies, >= 0.9 < 1.0

Report them like this:

   Found Explored Package  FluentAssertions 4.19.2
   1 packages in resolution.
     - FluentAssertions, 4.19.2
   4 requirements left
     - d:\Users\agross\Projekte\GROSSWEBER\Kunden\heco\heco\comwork\tools\paket.dependencies, FakeItEasy >= 3.3 < 4.0
     - d:\Users\agross\Projekte\GROSSWEBER\Kunden\heco\heco\comwork\tools\paket.dependencies, System.Windows.Interactivity >= 3.0 < 4.0
     - d:\Users\agross\Projekte\GROSSWEBER\Kunden\heco\heco\comwork\tools\paket.dependencies, Interop.CLMGRLib >= 2.0 < 3.0
     - d:\Users\agross\Projekte\GROSSWEBER\Kunden\heco\heco\comwork\tools\paket.dependencies, Machine.Specifications >= 0.9 < 1.0

@agross agross requested a review from forki May 18, 2017 20:43
@matthid
Copy link
Member

matthid commented May 18, 2017

Note that this is already fixed in Paket 5...

@agross
Copy link
Contributor Author

agross commented May 18, 2017

Alright, I didn't bother looking because I was trying to get a grip on #2342.

@forki forki merged commit 2ab7d37 into fsprojects:hotfix May 23, 2017
@agross agross deleted the report-package-names branch June 14, 2017 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@forki forki Awaiting requested review from forki

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@agross @matthid @forki