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

Adding a CONTRIBUTING document #437

Merged
merged 1 commit into from
Sep 9, 2015
Merged

Adding a CONTRIBUTING document #437

merged 1 commit into from
Sep 9, 2015

Conversation

@randomcamel
Copy link
Contributor

👍 Slick.

@stuartpreston
Copy link

Excuse my naivety - why must all Chef Provisioning resources implement :create and :destroy actions?
What if 'create' is the wrong verb (e.g. deploy a CF template, or in my case an Azure Resource Manager one)

@tyler-ball
Copy link
Contributor Author

@stuartpreston I think create is actually the wrong word - I would much more prefer converge as the default action. As you say, a Chef Resource may model an object which isn't actually created just interacted with. But the use of create as the default action goes wayyyyyyyy back.

If Chef Provisioning Azure should use some other actions, then we could update the CONTRIBUTING.md document there to say what the minimum set of actions the resources should respond to.

The goal here is not to force all Provisioning Resources to conform to a too-simple API, but rather to define a set of criteria so Resource authors know when their Resource is complete.

tyler-ball added a commit that referenced this pull request Sep 9, 2015
Adding a CONTRIBUTING document
@tyler-ball tyler-ball merged commit 0e48c1c into master Sep 9, 2015
@tyler-ball tyler-ball deleted the tball/contributing branch September 9, 2015 18:48
bglimepoint pushed a commit to bglimepoint/chef-provisioning that referenced this pull request Oct 29, 2018
@tas50 tas50 added Aspect: Documentation How do we use this project? and removed Type: Documentation labels Dec 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Aspect: Documentation How do we use this project?
Development

Successfully merging this pull request may close these issues.

5 participants