-
Notifications
You must be signed in to change notification settings - Fork 306
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 module paths to use open-policy-agent #305
Conversation
This is part of open-policy-agent/opa#2387. Signed-off-by: Torin Sandall <torinsandall@gmail.com>
Used the following find command arguments: find ./docs -type f -name "*.md" -exec sed -i= s/github\.com\\/instrumenta\\/conftest/github\.com\\/open-policy-agent\\/conftest/g {} \; Signed-off-by: Torin Sandall <torinsandall@gmail.com>
…ates Ran a quick grep to find other instances of github.com/instrumenta/conftest and these seemed to be the important ones to update. Note there are still references back to github.com/instrumenta in but those should be fine. Also, the image repository is still instrumenta/conftest but we can update those once CI is running. Signed-off-by: Torin Sandall <torinsandall@gmail.com>
🌋 📿 let's see what happens! |
It's not clear to me from the circle/acceptance.bats output what went wrong though this line seems likely to have caused it: https://github.com/instrumenta/conftest/blob/d58f240b6dd7f88b07592fa4727455dbe999bd13/acceptance.bats#L277 |
@tsandall the command is trying to pull from https://raw.githubusercontent.com/open-policy-agent/conftest/master/examples/compose/policy/deny.rego which (currently returns a $ conftest test --update https://raw.githubusercontent.com/open-policy-agent/conftest/master/examples/compose/policy/deny.rego examples/compose/docker-compose.yml
This should go away when conftest is actually migrated to the OPA org. |
Just so we're not deadlocked on each other, I'm OK to merge this in whenever you want to make the move official! |
I've moved the repo. We can merge this and get testing. |
I'm going to close and reopen this PR to make sure that DCO is enabled. |
This is for open-policy-agent/opa#2387 (comment).