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

Edits on resource api readme #17

Merged
merged 2 commits into from
Jan 30, 2018

Conversation

clairecadman
Copy link
Contributor

No description provided.

@clairecadman clairecadman requested a review from DavidS January 30, 2018 00:18
README.md Outdated
@@ -46,6 +46,7 @@ Do you want to continue and make these changes to your module? Yes
2 files modified.

~/git/example$

Copy link
Contributor

Choose a reason for hiding this comment

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

?

README.md Outdated
@@ -72,7 +73,7 @@ The three generated files are the type, the actual implementation, and unit test

### Writing the Type

The type contains the shape of your resources. The template renders the minimally necessary `name`, and `ensure` attributes. You can modify their description and the name's type to match your resource. Add more attributes as you see fit.
The type contains the shape of your resources. The template provides the necessary `name` and `ensure` attributes. You can modify their description and the name's type to match your resource. Add more attributes as you wish.
Copy link
Contributor

Choose a reason for hiding this comment

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

"... as you need."? Developers are pretty much on their own from here on, and need to make the changes they require for their goals.

We'll need to add more info on what/how they can do this later.

@DavidS
Copy link
Contributor

DavidS commented Jan 30, 2018

Except for two minor things this is good to go.

@DavidS DavidS merged commit 21546c2 into puppetlabs:master Jan 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants