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

update common Gemfile for chef11+12 #91

Merged
merged 3 commits into from
May 12, 2015
Merged

update common Gemfile for chef11+12 #91

merged 3 commits into from
May 12, 2015

Conversation

arlimus
Copy link
Member

@arlimus arlimus commented May 11, 2015

No description provided.

arlimus added 3 commits May 12, 2015 01:40
for the time being, we still have to support OS with legacy ruby, e.g. Ubuntu 12.04. So allow it to run all code correctly.

Signed-off-by: Dominik Richter <[email protected]>
Unfortunately we have to disable it at this point, if we don't want to risk writing more functions to cover the same space. Even if the buildup structure of the hashmap is completely removed from these functions (thus reducing the total number of assignments considerably), we are left with a slightly over AbcSize for get_macs, which is due to the OS-checking inside. So ignore it unless we find a better solution

Signed-off-by: Dominik Richter <[email protected]>
@arlimus arlimus mentioned this pull request May 12, 2015
@chris-rock
Copy link
Member

thanks for the lint improvements

chris-rock added a commit that referenced this pull request May 12, 2015
update common Gemfile for chef11+12
@chris-rock chris-rock merged commit 21e5e95 into master May 12, 2015
@chris-rock chris-rock deleted the update-common branch May 12, 2015 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants