-
Notifications
You must be signed in to change notification settings - Fork 11
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
Shallow cloning fails when running on pull request #34
Comments
Seems like grep should consider exclude |
@guitarrapc I think that should work yes! |
By the way, can you clarify what happen when calling curl without curl -X POST --location "https://circleci.com/api/v2/project/gh/storybookjs/storybook/pipeline" \
-H "Content-Type: application/json" \
-H "Circle-Token: $CIRCLE_CI_TOKEN" \
-d '{
"branch": "pull/19768",
"parameters": {
"workflow": "pr"
}
}' |
@guitarrapc I get a 400: {
"message" : "Branch not found"
} |
Also the API documentation says to use: https://circleci.com/docs/api/v2/index.html#operation/triggerPipeline |
Thank you! I will fix it. |
note: Before merging PR, I've try reproduce issue at my test lab but error not happen. Testing with following success.
|
ok, reproduced.
version: 2.1
orbs:
git-shallow-clone: guitarrapc/[email protected]
jobs:
test:
docker:
- image: cimg/base:stable
steps:
- git-shallow-clone/checkout_advanced
- run: echo "this is the test job"
workflows:
build_and_test:
jobs:
- test |
@kasperpeulen Hi, I've released v2.4.1 and confirm fixed on my lab. version: 2.1
orbs:
git-shallow-clone: guitarrapc/[email protected]
jobs:
test:
docker:
- image: cimg/base:stable
steps:
- git-shallow-clone/checkout_advanced
- run: echo "this is the test job"
workflows:
build_and_test:
jobs:
- test Thanks, |
Thanks! Trying it out now :) |
When I run the API for a pull request:
I get an error in the shallow clone:
This line seems to be the problem:
https://github.com/guitarrapc/git-shallow-clone-orb/blob/master/src/commands/checkout_advanced.yml#L103
I also tried calling the API with:
"branch": "pull/19768"
instead of"branch": "pull/19768"
but that gives me a branch not found error. (400)The text was updated successfully, but these errors were encountered: