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.

Sign up for GitHub

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

Jump to bottom

expose GITHUB_REPOSITORY_OWNER. #378

Merged
merged 1 commit into from
Mar 20, 2020
Merged

expose GITHUB_REPOSITORY_OWNER. #378

merged 1 commit into from
Mar 20, 2020

Conversation

TingluoHuang
Copy link
Member

@TingluoHuang TingluoHuang commented Mar 19, 2020 •
edited
Loading

@TingluoHuang TingluoHuang force-pushed the users/tihuang/repositoryowner branch from 7c1bf0c to e3b3744 Compare March 20, 2020 14:20
@TingluoHuang TingluoHuang merged commit 178a618 into master Mar 20, 2020
@TingluoHuang TingluoHuang deleted the users/tihuang/repositoryowner branch March 20, 2020 17:02
@gitfool
Copy link

gitfool commented Apr 4, 2020

@TingluoHuang @ericsciple I ran a build with the latest hosted images and runner v2.168.0. I now see the CI and GITHUB_JOB environment variables but I don't see GITHUB_REPOSITORY_OWNER:

Is there something I'm missing? 🤔

@gitfool
Copy link

gitfool commented Apr 10, 2020

Please let me know if GITHUB_REPOSITORY_OWNER should be visible, otherwise I'll need to remove support for it from my Cake PR.

@TingluoHuang
Copy link
Member Author

@gitfool I just tried on my account, I can see the env when I printenv|sort

GITHUB_REPOSITORY=TingluoHuang/main
GITHUB_REPOSITORY_OWNER=TingluoHuang

What's your repository name that not has this env? I can check to see whether this is due to delay in deployment.

@gitfool
Copy link

gitfool commented Apr 10, 2020

@TingluoHuang see my links above showing build logs with env vars for gitfool/Cake.Dungeon.

@gitfool
Copy link

gitfool commented Apr 10, 2020

@TingluoHuang kicked off another build, same runner v2.168.0 but this time I see GITHUB_REPOSITORY_OWNER. Weird but answers my question. Thanks anyway!

AdamOlech pushed a commit to antmicro/runner that referenced this pull request Jan 28, 2021
@TingluoHuang
expose GITHUB_REPOSITORY_OWNER. (actions#378)
ea2931d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ericsciple ericsciple ericsciple approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@TingluoHuang @gitfool @ericsciple