Thomas Powell

Results 128 comments of Thomas Powell

Seeing that @clintoncwolfe is assigned to the internal issue.

@dafyddcrosby any more movement on this one?

@johnmccrae is this meant to be a draft PR?

@Babar can you add a couple cases in spec/unit/plugins/linux/lspci_spec.rb to match the additional regex component specifically?

@aaronclarke do you have any capacity to propose better error handling and/or documentation in a PR? Seems like a beneficial fix, but might also get lost in the priority shuffle...

@williamtheaker can you add the exact version and possibly a link to the Apple source inside the description for completeness?

@williamtheaker upon further review, perhaps this functionality should go straight to getting a deprecation warning instead so that you will get a direct admonition when using. macOS 11 is EOL...

> > @williamtheaker upon further review, perhaps this functionality should go straight to getting a deprecation warning instead so that you will get a direct admonition when using. macOS 11...

> I couldn't find any examples of previously retired resources for reference. Should I just add `Chef::Log.warn("This resource is deprecated and will be removed in a future #{ChefUtils::Dist::Infra::PRODUCT} release.")` to...

> I realized [previous resource deprecations](https://docs.chef.io/chef_deprecations_client/#all-deprecations) required RFCs (now design proposals) so I've opened one to deprecate this resource in #14278 and will update this PR when (🤞) the proposal...