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

Skipped context check on OSX #2453

Merged
merged 1 commit into from Feb 12, 2019
Merged

Skipped context check on OSX #2453

merged 1 commit into from Feb 12, 2019

Conversation

svalleru
Copy link
Contributor

@svalleru svalleru commented Feb 11, 2019

Regex is causing some issues on OSX , so, skipping the check on OSX for now until the new go binary is out which will solve this for good

Related to: #1948


This change is Reviewable

Regex is causing some issues on OSX , so, skipping the check on OSX for now until the new go binary is out which will solve this for good
@jlewi
Copy link
Contributor

jlewi commented Feb 12, 2019

@svalleru could we cherry pick this into the 0.4 branch as well?

/lgtm
/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jlewi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit c1165e5 into kubeflow:master Feb 12, 2019
@svalleru
Copy link
Contributor Author

svalleru commented Feb 12, 2019

@svalleru could we cherry pick this into the 0.4 branch as well?

yes, incorporated this into #2314 - please review

kimwnasptd pushed a commit to arrikto/kubeflow that referenced this pull request Feb 13, 2019
Regex is causing some issues on OSX , so, skipping the check on OSX for now until the new go binary is out which will solve this for good
saffaalvi pushed a commit to StatCan/kubeflow that referenced this pull request Feb 11, 2021
Regex is causing some issues on OSX , so, skipping the check on OSX for now until the new go binary is out which will solve this for good
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants