-
Notifications
You must be signed in to change notification settings - Fork 572
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
feature request: ability to configure the github domain of the default git context (for Github Enterprise users) #326
Comments
I think we could instead extract this from the runner context. Can you add this step in your workflow and tell me if your domain is found in the context? Thanks. |
From what I see, |
@crazy-max Thanks a ton for your fast and promising replies. Those values are actually set - verified with your
So basically
But maybe (just guessing) it is overwritten here and should come out of the env var or the context instead (just as your first post suggests): build-push-action/src/context.ts Line 40 in 6a12baa
|
Yes just wanted to be sure _defaultContext = `${github.server_url}/${github.context.repo.owner}/${
github.context.repo.repo
}.git#${github.context?.ref?.replace(/^refs\//, '')}`; |
@digitalkram I'm having a similiar issue. I saw this was just released. But still have the same issue even though I set the env in my workflow.yml. Is there a syntax I'm missing? I've tried both ways of setting this Setting the environment variable on the specific action
Setting the environment variable on the whole workflow
|
@bondezbond
Not yet shipped but you can test with |
sigh I need my coffee already :) |
The domain of the default context is hard-coded to
github.com
:build-push-action/src/context.ts
Line 40 in 6a12baa
For users of Github Enterprise running on a custom domain like
git.mycompany.com
it would be great if this would be configurable like:The text was updated successfully, but these errors were encountered: