Hi Dan,

Sure you are right. I will create a separate pull request with the changes
and updated documentation as well as test.

cheers,
inny


On Thu, May 22, 2014 at 4:42 PM, Dan Carley <notifications@github.com>wrote:

> In manifests/init.pp:
>
> > @@ -22,6 +22,7 @@
> >    $package_ensure = present,
> >    $config = {},
> >    $repo = true,
> > +  $key_server = 'keys.gnupg.net',
>
> I know this has already been merged, but I think this should default to
> undef so that we don't stomp on the upstream defaults in puppetlabs-apt.
> Especially as their interface has changed slightly in
> puppetlabs/puppetlabs-apt@5790341<https://github.com/puppetlabs/puppetlabs-apt/commit/57903418909c3c3344a4c4fa41c054e954451227>
>
> Would you mind raising a separate pull request?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/alphagov/puppet-aptly/pull/10/files#r12956908>
> .
>