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.

Sign up for GitHub

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

Jump to bottom

Created the resource 'OS Deployment Plan' #213

Merged
merged 1 commit into from
Mar 14, 2017
Merged

Created the resource 'OS Deployment Plan' #213

merged 1 commit into from
Mar 14, 2017

Conversation

ricardogpsf
Copy link
Member

Description

The resource OS Deployment Plan was created. This resource is important for use with server profile, for example.

Issues Resolved

Fixes #212

Check List

  • New functionality includes testing.
    • All tests pass ($ rake test).
  • New functionality has been documented in the README if applicable.
    • New functionality has been thoroughly documented in the examples (please include helpful comments).
  • Changes are documented in the CHANGELOG.

@ricardogpsf ricardogpsf force-pushed the fix/os_deployment_plan_resource branch from 8ccbdae to 9c763f4 Compare March 14, 2017 17:02
@marikrg
Copy link

marikrg commented Mar 14, 2017

👍 Looks good for me

fgbulsoni
fgbulsoni approved these changes Mar 14, 2017
View reviewed changes
Copy link
Contributor

@fgbulsoni fgbulsoni left a comment

Choose a reason for hiding this comment

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

Looks good, simple resource but will be plenty useful.

@ricardogpsf ricardogpsf merged commit cad8705 into master Mar 14, 2017
@aalexmonteiro aalexmonteiro deleted the fix/os_deployment_plan_resource branch March 20, 2017 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fgbulsoni fgbulsoni fgbulsoni approved these changes

@aalexmonteiro aalexmonteiro Awaiting requested review from aalexmonteiro

Assignees
No one assigned
Labels
new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@ricardogpsf @marikrg @fgbulsoni @jsmartt