Skip to content

improve error message when Package.resolved cannot be loaded #3721

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

Merged
merged 2 commits into from
Sep 8, 2021

Conversation

tomerd
Copy link
Contributor

@tomerd tomerd commented Sep 7, 2021

motivation: provide clear and actionable information when trying to load pacakge.resolved file from a non-supported version, or otherwise malformed.

changes:

  • add contextual information to error message
  • add few tests

@tomerd
Copy link
Contributor Author

tomerd commented Sep 7, 2021

needs #3717 first

@tomerd tomerd force-pushed the fix/serialization-error-message branch 2 times, most recently from 05753f8 to 385eeb8 Compare September 7, 2021 00:35
@tomerd
Copy link
Contributor Author

tomerd commented Sep 7, 2021

@swift-ci please smoke test

@tomerd tomerd added the ready Author believes the PR is ready to be merged & any feedback has been addressed label Sep 7, 2021
@tomerd tomerd force-pushed the fix/serialization-error-message branch from 385eeb8 to 1752166 Compare September 7, 2021 23:54
@tomerd
Copy link
Contributor Author

tomerd commented Sep 7, 2021

@swift-ci please smoke test

@tomerd tomerd force-pushed the fix/serialization-error-message branch from 1752166 to 7e28c6b Compare September 8, 2021 00:21
motivation: provide clear and actionable information when trying to load pacakge.resolved file from a non-supported version, or otherwise malformed.

changes:
* add contextual information to error message
* add few tests
@tomerd tomerd force-pushed the fix/serialization-error-message branch from 7e28c6b to 1585f67 Compare September 8, 2021 00:23
@tomerd
Copy link
Contributor Author

tomerd commented Sep 8, 2021

@swift-ci please smoke test

@tomerd tomerd force-pushed the fix/serialization-error-message branch from 1585f67 to 8b2a406 Compare September 8, 2021 00:26
@tomerd
Copy link
Contributor Author

tomerd commented Sep 8, 2021

@swift-ci please smoke test

@tomerd tomerd force-pushed the fix/serialization-error-message branch from 8b2a406 to ed46762 Compare September 8, 2021 03:58
@tomerd
Copy link
Contributor Author

tomerd commented Sep 8, 2021

@swift-ci please smoke test

@tomerd tomerd merged commit 4f45968 into swiftlang:main Sep 8, 2021
mattt pushed a commit to mattt/swift-package-manager that referenced this pull request Sep 16, 2021
…ng#3721)

* improve error message when Package.resolved cannot be loaded

motivation: provide clear and actionable information when trying to load pacakge.resolved file from a non-supported version, or otherwise malformed.

changes:
* add contextual information to error message
* add few tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready Author believes the PR is ready to be merged & any feedback has been addressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants