Skip to content

database_user gives the wrong deprecation warning #345

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 30, 2013
Merged

database_user gives the wrong deprecation warning #345

merged 1 commit into from
Oct 30, 2013

Conversation

igalic
Copy link
Contributor

@igalic igalic commented Oct 30, 2013

This appears to have been a bad copy/paste error.
Reported by @vicinus, this should fix #344

This appears to have been a bad copy/paste error.
Reported by @vicinus, this should fix #344
@igalic igalic merged commit 1afa657 into puppetlabs:master Oct 30, 2013
@igalic igalic deleted the copy-paste-error branch October 30, 2013 22:27
@@ -7,7 +7,7 @@
newparam(:name, :namevar=>true) do
desc "The name of the user. This uses the 'username@hostname' or username@hostname."
validate do |value|
Puppet.warning("database has been deprecated in favor of mysql_database.")
Puppet.warning("database has been deprecated in favor of mysql_user.")
Copy link
Contributor

Choose a reason for hiding this comment

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

We should probably make this database_user has been, :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

_ it's been very late yesternight. sorry.

@igalic
Copy link
Contributor Author

igalic commented Oct 31, 2013

let's try this again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants