added Puppetfile.lock into gitignore; Some cleaning is a good practice; Please merge it into master

[ghost]

No description provided.

[dgoodlad]

👎 Puppetfile.lock is required, primarily for bootstrapping. Why do you want to remove it?

[ghost]

required for bootstrapping?

Just created when puppet is processed.

[dgoodlad]

Without Puppetfile.lock, same as Gemfile.lock, you can end up with a dependency cycle – think authentication – when you bootstrap a brand new machine.

Again, why do you think this file should be removed?

[ghost]

isn't it created by Puppet for the first time (even for bootstrapping) ?

[dgoodlad]

@julienlavergne it may be regenerated if you've changed Puppetfile (by librarian-puppet, not by Puppet), but that shouldn't be happening when bootstrapping

[ghost]

@dgoodlad effectively using librarian-puppet prevents it.