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

feature: bring back profile check #150

Merged
merged 3 commits into from
Oct 26, 2015
Merged

feature: bring back profile check #150

merged 3 commits into from
Oct 26, 2015

Conversation

arlimus
Copy link
Contributor

@arlimus arlimus commented Oct 26, 2015

Check a profile; exit code 1 if the check finds any errors; 0 otherwise; print warnings

res
end

def check # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add a short description of expected return values?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point, thank you for pointing this out!

@chef-delivery
Copy link
Contributor

This PR has passed 'Verify' and is ready for review and approval!
Use: '@delivery approve' when code review is complete.

@chris-rock
Copy link
Contributor

thank you very much @arlimus

@chris-rock
Copy link
Contributor

@delivery approve

chef-delivery added a commit that referenced this pull request Oct 26, 2015
Merged change 217ee46b-1a88-46a9-86a5-f65209c2d3eb

From review branch profile-check into master

Signed-off-by: chartmann <chartmann@chef.io>
@chef-delivery chef-delivery merged commit a981369 into master Oct 26, 2015
@chef-delivery
Copy link
Contributor

Change: 217ee46b-1a88-46a9-86a5-f65209c2d3eb approved by: @chris-rock

@chef-delivery chef-delivery deleted the profile-check branch October 26, 2015 11:12
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.

None yet

3 participants