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

CI doesn't function for forks #144

Closed
3 tasks done
oflatt opened this issue Jun 4, 2024 · 4 comments
Closed
3 tasks done

CI doesn't function for forks #144

oflatt opened this issue Jun 4, 2024 · 4 comments
Labels
backlog We hope to work on this in the future bug Something isn't working good-first-issue Good for newcomers

Comments

@oflatt
Copy link
Contributor

oflatt commented Jun 4, 2024

Before opening, please confirm:

Bug Category

cedar-example-use-cases

Describe the bug

Currently, the CI for cedar-examples fails for forks.
See this example run: https://github.com/cedar-policy/cedar-examples/actions/runs/9371870708/job/25802115924?pr=142
AFAICT, this is because it tries to fetch from the main repo instead of the fork.
This is blocking two of my branches (though we can also just give me permissions for this repo).

Expected behavior

I would expect CI to succeed for branches on forks.

Reproduction steps

  1. Fork this repository.
  2. Push a branch.
  3. Ci fails

Code Snippet

No response

Log output

No response

Additional configuration

No response

Operating System

No response

Additional information and screenshots

No response

@oflatt oflatt added bug Something isn't working pending-triage Hasn't been triaged yet labels Jun 4, 2024
@shaobo-he-aws shaobo-he-aws added good-first-issue Good for newcomers and removed pending-triage Hasn't been triaged yet labels Jun 4, 2024
@john-h-kastner-aws
Copy link
Contributor

john-h-kastner-aws commented Jun 4, 2024

See #90. We had some other issue with builds from forks previously.

Perhaps that PR only fixed builds coming from cedar-policy/cedar?

@andrewmwells-amazon
Copy link
Contributor

This seems to be working for forks on my personal GH account:
#146
#147

@andrewmwells-amazon
Copy link
Contributor

Hopefully fixed with #149

@andrewmwells-amazon
Copy link
Contributor

#149

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog We hope to work on this in the future bug Something isn't working good-first-issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants