-
Notifications
You must be signed in to change notification settings - Fork 40
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
1045 repo 4.0 #1087
1045 repo 4.0 #1087
Conversation
89e66f7
to
0b3c451
Compare
0b3c451
to
db898e1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, just some minor comments.
@@ -439,6 +321,25 @@ public void DownloadPackage(IPackageIdentifier package, string destination, Canc | |||
} | |||
} | |||
|
|||
private void Auth() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we call it something like, AttachAuthentication?
PackageDef.SaveManyTo(stream, ConvertToPackageDef(compatibleWith)); | ||
stream.Seek(0, 0); | ||
string data = new StreamReader(stream).ReadToEnd(); | ||
["type"] = @class, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Type and class are not the same. It should just be class
This change is part of OpenTAP version |
Use repo client and target repo 4.0 API to support authentication
Closes #1045