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

Unclear error message for issues in lock file #1247

Closed
TeaDrivenDev opened this Issue Nov 22, 2015 · 5 comments

Comments

Projects
None yet
2 participants
@TeaDrivenDev
Contributor

TeaDrivenDev commented Nov 22, 2015

When there is something in paket.dependencies that Paket doesn't understand, it tells you exactly where, and what the problem is. For an error in paket.lock, I only got

Paket failed with:
    Input string was not in a correct format.

This should at least say that it's about the lock file, so the user knows where to start looking.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Nov 22, 2015

Member

How did the wrong lock file look like?

Member

forki commented Nov 22, 2015

How did the wrong lock file look like?

@TeaDrivenDev

This comment has been minimized.

Show comment
Hide comment
@TeaDrivenDev

TeaDrivenDev Nov 22, 2015

Contributor

I had tried to allow a "minimum" version at the top level:

NUGET
  remote: https://nuget.org/api/v2
  specs:
    FAKE (>= 4.9.3)
Contributor

TeaDrivenDev commented Nov 22, 2015

I had tried to allow a "minimum" version at the top level:

NUGET
  remote: https://nuget.org/api/v2
  specs:
    FAKE (>= 4.9.3)
@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Nov 23, 2015

Member

I just added this as an integration test and it now shows a better error message, but why do you change the lockfile manually?

Member

forki commented Nov 23, 2015

I just added this as an integration test and it now shows a better error message, but why do you change the lockfile manually?

@forki forki closed this in 40c90e6 Nov 23, 2015

@TeaDrivenDev

This comment has been minimized.

Show comment
Hide comment
@TeaDrivenDev

TeaDrivenDev Nov 23, 2015

Contributor

Because

200_s

;-)

Contributor

TeaDrivenDev commented Nov 23, 2015

Because

200_s

;-)

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Nov 23, 2015

Member

yeah, lock files are usually not edited manually.

Member

forki commented Nov 23, 2015

yeah, lock files are usually not edited manually.

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