Skip to content

Conversation

JacobTanenbaum
Copy link
Contributor

QA is having issues using the script because it errors on the index function.

Bug: 1429398 Link

QA is having issues using the script because it errors on the index function.

Bug: 1429398
@JacobTanenbaum
Copy link
Contributor Author

@knobunc @stevekuznetsov PTAL

@stevekuznetsov
Copy link
Contributor

Seems reasonable as long as you don't feel it is more appropriate to write a short Go or Python script that uses the stdlib to do JSON manipulation.

Copy link
Contributor

@knobunc knobunc left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks Jake!

@knobunc
Copy link
Contributor

knobunc commented Apr 10, 2017

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to eb5f106

@openshift-bot
Copy link
Contributor

openshift-bot commented Apr 10, 2017

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin/303/) (Base Commit: f65accd) (Image: devenv-rhel7_6131)

@openshift-bot openshift-bot merged commit d128020 into openshift:master Apr 10, 2017
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.

4 participants