From 8c3a498bdd9458c1227b495c03c99f3aaa700061 Mon Sep 17 00:00:00 2001
From: Paula Muir <paula@puppet.com>
Date: Mon, 21 Jun 2021 11:03:07 +0100
Subject: [PATCH] (IAC-1430) - Minor docs updating

---
 lib/puppet/functions/mysql/normalise_and_deepmerge.rb | 9 +++++++--
 manifests/client.pp                                   | 4 ----
 types/options.pp                                      | 2 ++
 3 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/lib/puppet/functions/mysql/normalise_and_deepmerge.rb b/lib/puppet/functions/mysql/normalise_and_deepmerge.rb
index c2f5880ab..bb5f884f3 100644
--- a/lib/puppet/functions/mysql/normalise_and_deepmerge.rb
+++ b/lib/puppet/functions/mysql/normalise_and_deepmerge.rb
@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
-# @summary Recursively merges two or more hashes together, normalises keys with differing use of dashesh and underscores,
-#   then returns the resulting hash.
+# @summary Recursively merges two or more hashes together, normalises keys with differing use of dashes and underscores.
 #
 # @example
 #   $hash1 = {'one' => 1, 'two' => 2, 'three' => { 'four' => 4 } }
@@ -15,6 +14,12 @@
 # - When there are conficting uses of dashes and underscores in two keys (which mysql would otherwise equate), the rightmost style will win.
 #
 Puppet::Functions.create_function(:'mysql::normalise_and_deepmerge') do
+  # @param args
+  #   Hash to be normalised
+  #
+  # @return hash
+  #   The given hash normalised
+  #
   def normalise_and_deepmerge(*args)
     if args.length < 2
       raise Puppet::ParseError, _('mysql::normalise_and_deepmerge(): wrong number of arguments (%{args_length}; must be at least 2)') % { args_length: args.length }
diff --git a/manifests/client.pp b/manifests/client.pp
index e1bd10e05..402d30e9e 100644
--- a/manifests/client.pp
+++ b/manifests/client.pp
@@ -16,10 +16,6 @@
 #   Whether the MySQL package should be present, absent, or a specific version. Valid values are 'present', 'absent', or 'x.y.z'.
 # @param package_manage
 #   Whether to manage the MySQL client package. Defaults to `true`.
-# @param service_name
-#   The name of the MySQL server service. Defaults are OS dependent, defined in 'params.pp'.
-# @param service_provider
-#   The provider to use to manage the service. For Ubuntu, defaults to 'upstart'; otherwise, default is undefined.
 # @param package_name
 #   The name of the MySQL client package to install.
 #
diff --git a/types/options.pp b/types/options.pp
index ab08945a4..71a8f5923 100644
--- a/types/options.pp
+++ b/types/options.pp
@@ -1,3 +1,5 @@
+# @summary A hash of options structured like the override_options, but not merged with the default options.
+# Use this if you don’t want your options merged with the default options.
 type Mysql::Options = Hash[
   String,
   Hash,