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.

Sign up for GitHub

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

Jump to bottom

Skip SElinux recipe on non linux OSes. #223

Merged
merged 1 commit into from
Sep 25, 2015
Merged

Skip SElinux recipe on non linux OSes. #223

merged 1 commit into from
Sep 25, 2015

Conversation

kamaradclimber
Copy link
Contributor

No description provided.

@codecov-io
Copy link

Current coverage is 68.32%

Merging #223 into master will not affect coverage as of 00c1c06

@@            master    #223   diff @@
======================================
  Files            6       6       
  Stmts          322     322       
  Branches         0       0       
  Methods          0       0       
======================================
  Hit            220     220       
  Partial          0       0       
  Missed         102     102

Review entire Coverage Diff as of 00c1c06

Powered by Codecov. Updated on successful CI builds.

johnbellone added a commit that referenced this pull request Sep 25, 2015
@johnbellone
Merge pull request #223 from criteo-forks/selinux
ce6fd39 
Skip SElinux recipe on non linux OSes.
@johnbellone johnbellone merged commit ce6fd39 into sous-chefs:master Sep 25, 2015
@kamaradclimber kamaradclimber deleted the selinux branch September 26, 2015 05:54
@kamaradclimber
Copy link
Contributor Author

thanks for merging this, I am not sure it is sufficient to fix the windows setup so I may provide other patches in this direction.

@lock
Copy link

lock bot commented Apr 25, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Apr 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kamaradclimber @codecov-io @johnbellone