dave-q
dave-q
> This is a valid ask. Could you please share the branch name where you have pushed changes so we can discuss further? I haven't pushed a branch yet, just...
@Xorima So after a bit of fiddling I've got the test's passing. The code in the PR is all working but in my opinion I need to give it one...
@bmhughes @rlaveycal Thanks for taking the time to review my PR. I've updated it with your suggestions and all the tests are passing. I have a couple of questions. 1....
@lamont-granquist @bmhughes I appreciate the feedback. You've both made a couple of comments that seemed to be based around the thought that the only place we are using the new...
> I know the sticks in the mud hate it but is there any reason those recipes can't go away and be replaced by a resource? A lot of them...
I can definitely see the benefits of that model for sure. I'm to familiar with the use cases of chef. I know at my work we have our own internal...
So I've taken a lot of what you both suggested on board @bmhughes @lamont-granquist I noticed that with a little refactoring, the install resource could do all the new work...
I'm happy to put in a PR for this myself :)
is this still in need of doing?
Is there any info on this? The documentation recommends using this over system.management but it appears we can't for 4.7.2 applications? Some guidance would be appreciated