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

Include package name in verbose report #2349

Merged
merged 1 commit into from May 23, 2017

Conversation

Projects
None yet
3 participants
@agross
Contributor

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

@matthid

This comment has been minimized.

Show comment
Hide comment
@matthid

matthid May 18, 2017

Member

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

Member

matthid commented May 18, 2017

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

@agross

This comment has been minimized.

Show comment
Hide comment
@agross

agross May 18, 2017

Contributor

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

Contributor

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

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details

@agross agross deleted the agross:report-package-names branch Jun 14, 2017

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