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

Need to validate Manifest for existence of required Libraries property #303

Open
raquelsa opened this issue Jul 12, 2018 · 1 comment
Open

Need to validate Manifest for existence of required Libraries property #303

raquelsa opened this issue Jul 12, 2018 · 1 comment
Labels
P2

Comments

@raquelsa
Copy link
Contributor

We are getting null ref exceptions if Manifest does not have Libraries propertiy specified.
We should add validation for Manifest for required properties.

Trying to restore a LibMan.json without Libraries property will lead to the messages in the output window:

Restore operation started...
Restoring libraries for project Core20_AllRestoreErrors
Restore operation completed
Time Elapsed: 00:00:00.0062925
========== Finished ==========
@jodavis jodavis added the P2 label Jul 12, 2018
@RobJohnston
Copy link
Contributor

Duplicate of #249, created 3 weeks prior, tagged with "P3" and "bug".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@raquelsa @RobJohnston @jodavis