Skip to content
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

(MODULES-2464) Simplify query for sp_configure check #144

Merged
merged 1 commit into from
Sep 2, 2015
Merged

(MODULES-2464) Simplify query for sp_configure check #144

merged 1 commit into from
Sep 2, 2015

Conversation

cyberious
Copy link
Contributor

  • With the wrapping query it was returning no error due to success of table creation and
    did not properly flag the error.

@ferventcoder
Copy link
Contributor

Git commit nitpick - I don't really feel I get a good explanation of what changed in the commit message. Would you mind explaining it better why you no longer need the section of code that was removed?

@ferventcoder ferventcoder changed the title (MODULES-2464) Simplify query for sp_configure check [DO NOT MERGE](MODULES-2464) Simplify query for sp_configure check Aug 31, 2015
@ferventcoder
Copy link
Contributor

You also have specs to fix up.

- Due to the success of table creation and insertion the query executes successfully
  even though when ran through SSMS it properly throws the error. Fallout from ruby
  refactor
@cyberious cyberious changed the title [DO NOT MERGE](MODULES-2464) Simplify query for sp_configure check (MODULES-2464) Simplify query for sp_configure check Aug 31, 2015
@cyberious
Copy link
Contributor Author

@ferventcoder updated and passing

ferventcoder added a commit that referenced this pull request Sep 2, 2015
(MODULES-2464) Simplify query for sp_configure check
@ferventcoder ferventcoder merged commit fbaabc1 into puppetlabs:master Sep 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants