Skip to content

Conversation

@v-anshie
Copy link
Contributor

  • Added templates for bug reports, feature requests, and pull requests to adhere to adhere with Github's recommended community standards.
  • Added support for owner/group in the plist resource. Allows for plist files to be created under a specific owner. Defaults to root/wheel for compatibility with earlier versions of the cookbook. (Issue #51)
  • Added support for setting the mode property when creating a plist using the plist resource. This allows control over setting the file permissions. (Issue #51)

jazaval and others added 18 commits December 6, 2018 14:04
* Update issue templates

Add in new issue templates for bug reports and feature requests. 

Bug reports will have a basic template covering "description", "error messages", and "chef exceptions"

Feature requests will have a basic template covering "description", and a "reasoning behind feature"

* fix spacing for bug report and feature request templates

* add pull request template

* update changelog with changes

* respond to code review comments and change header to ##

* respond to code review comments and use Justification and change header to ##

* move CONTRIBUTING and PULL_REQUEST_TEMPLATE into ISSUE_TEMPLATE directory as suggested in PR comments

* move github specific docs into github folder

* move shared docs into root
- Paired with Andre Shields and Chris Gilbert
Co-Authored-By: v-anshie <v-anshie@microsoft.com>
added description of user/group plist support.
Paired with Chris Gilbert, Andre Shields, Carrol Fifer
Add support for owner/group to plist resource
Co-Authored-By: v-anshie <v-anshie@microsoft.com>
@v-anshie v-anshie requested a review from chrismarlin December 21, 2018 19:07
@chrismarlin
Copy link
Contributor

It might be helpful to add unit or integration testing to validate the behavior of modifications to the user or group of an existing plist.

@v-anshie v-anshie reopened this Dec 27, 2018
@v-anshie v-anshie merged commit 64fe139 into master Dec 31, 2018
@jeroenj
Copy link

jeroenj commented Jan 2, 2019

Could it be that there was something wrong releasing this to the Chef supermarket? Or does merging this not necessarely mean that the new version is being released?

@jazaval
Copy link
Contributor

jazaval commented Jan 2, 2019

Sorry about that @jeroenj. 2.9.0 is now available on the Chef Supermarket. https://supermarket.chef.io/cookbooks/macos/versions/2.9.0

@jazaval jazaval deleted the release/2.9 branch January 2, 2019 18:10
@jeroenj
Copy link

jeroenj commented Jan 2, 2019

Thanks! 🙇‍♂️

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.

7 participants