search

chef / chef-apply

The ad-hoc execution tool for the Chef ecosystem.
https://www.chef.sh/
Apache License 2.0
14 stars 14 forks source link

[tech debt] group related classes together #65

Closed marcparadise closed 5 years ago

marcparadise commented 5 years ago

Description

This is a re-submit of #63, which was merged prematurely.

marcparadise commented 5 years ago

For those who haven't reviewed yet - when looking by commit, it's a lot cleaner - it shows that the bulk of the change is rename/renamespace/whitespace, eg https://github.com/chef/chef-apply/pull/65/commits/93e120a3835392a32108a52f0eee513cd8c0329b?utf8=%E2%9C%93&diff=unified&w=1

Viewing the full PR diff hides all of the renames that happened in lib/ and treats them like deleted/new files.

marcparadise commented 5 years ago

I'll bring this up to date so we can merge it.

marcparadise commented 5 years ago

This is up to date, will merge after verifying recent commits from master are correct post-rebase.