Merge pull request #11 from tobias-m/typos

fixed typos, linting warnings, errors and a wrong variable name inside g...

Author: alvagante

manifests/clone.pp

@@ -30,13 +30,14 @@
 #   Owner of the cloned repo.
 #
 define git::clone (
-  $ensure        = 'present',
   $source_url,
   $destination_dir,
+  $ensure        = 'present',
   $creates       = $destination_dir,
   $extra_options = '',
   $branch        = $git::default_branch,
   $owner         = $git::default_owner,) {
+
   include git
 
   validate_string($owner)
@@ -52,7 +53,7 @@
       }
     }
     'absent'  : {
-      file { "${destination_dir}":
+      file { $destination_dir:
         ensure  => 'absent',
         recurse => true,
         force   => true,

manifests/init.pp

@@ -143,8 +143,8 @@
   ### Managed resources
   if !defined(Package[$git::package]) {
     package { $git::package:
-      ensure  => $git::manage_package,
-      noop    => $git::noops,
+      ensure => $git::manage_package,
+      noop   => $git::noops,
     }
   }
 

manifests/reposync.pp

@@ -16,7 +16,7 @@
 #   present in the git_reposync script. Required.
 #
 # [*destination_dir*]
-#   Local directory where to sync the repository, As passed to the
+#   Local directory where to sync the repository. As passed to the
 #   git command present in the git_reposync script. Required.
 #
 # [*extra_options*]
@@ -35,13 +35,13 @@
 #   command (when autorun is enabled). Default: $destination_dir.
 #
 # [*pre_command*]
-#   Optional comman to execute before executing the git command.
+#   Optional command to execute before executing the git command.
 #   Note that this command is placed in the git_reposync script created
 #   by this define and it's executed every time this script is run (either
 #   manually or via Puppet). Default: ''
 #
 # [*post_command*]
-#   Optional comman to execute after executing the git command.
+#   Optional command to execute after executing the git command.
 #   Note that this command is placed in the git_reposync script created
 #   by this define and it's executed every time this script is run (either
 #   manually or via Puppet). Default: ''
@@ -132,8 +132,8 @@
 
   if $autorun == true {
     exec { "git_reposync_run_${name}":
-      command     => "${basedir}/git_reposync_${name}",
-      creates     => $creates,
+      command => "${basedir}/git_reposync_${name}",
+      creates => $creates,
     }
   }
 

templates/reposync/git_reposync-command.erb

@@ -25,6 +25,6 @@ fi
 
 <%= @post_command %>
 
-<% if @execute_on_change == true %>
+<% if @manage_execute_on_change == true %>
 fi
 <% end %>