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

Fix CI requiring original repo (#560) #561

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RPDeshaies
Copy link
Collaborator

βœ… Changes

  • fix: This should fix the CI failing on forks.

πŸŒ„ Context

I had noticed when fixing the issue of the cap on Numeric fields that the CI consistently failed. Looking from the build logs, I think the issue is that it looks for secrets related to the main repo in a way that seems hardcoded. This should fix that particular issue.

βœ… Changes

πŸŒ„ Context

πŸ”’Checklist

  • I tested my work on the feature environment

πŸ’… Examples

## βœ… Changes

<!-- Use prefixes: **chore**, **docs**, **feat**, **fix**, **refactor**,
**style** or **test** -->

- **fix**: This should fix the CI failing on forks.

## πŸŒ„ Context

I had noticed when fixing the issue of the cap on Numeric fields that
the CI consistently failed. Looking from the build logs, I think the
issue is that it looks for secrets related to the main repo in a way
that seems hardcoded. This _should_ fix that particular issue.
@glmdgrielson
Copy link
Contributor

That looks like it did a good!

@RPDeshaies RPDeshaies enabled auto-merge (squash) November 25, 2024 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants