Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove Notary from Repository constructor functions #1226

Merged
merged 1 commit into from
Sep 8, 2017

Conversation

tiborvass
Copy link
Contributor

Also fixes a small bug with SetLegacyVersions.

Signed-off-by: Tibor Vass teabee89@gmail.com

Copy link
Contributor

@riyazdf riyazdf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM


// targets should have 1 target, and it should be readable offline
targets, err := repo.ListTargets()
require.NoError(t, err)
require.Len(t, targets, 1)
require.Equal(t, "LICENSE", targets[0].Name)



Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

super non-blocking nit: odd spacing here

@tiborvass tiborvass force-pushed the change-constructor-names branch 2 times, most recently from c848281 to 00799c2 Compare September 7, 2017 22:16
Copy link
Contributor

@NathanMcCauley NathanMcCauley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Also fixes a small bug with SetLegacyVersions.

Signed-off-by: Tibor Vass <teabee89@gmail.com>
@tiborvass tiborvass force-pushed the change-constructor-names branch from 00799c2 to 87c5919 Compare September 8, 2017 00:15
Copy link
Contributor

@endophage endophage left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@endophage endophage merged commit 8a1de3c into notaryproject:master Sep 8, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants