-
Notifications
You must be signed in to change notification settings - Fork 282
Replace hardcoded /opt/gh-aw/ paths with GH_AW_HOME env var for self-hosted runner portability
#20143
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
base: main
Are you sure you want to change the base?
Replace hardcoded /opt/gh-aw/ paths with GH_AW_HOME env var for self-hosted runner portability
#20143
Changes from all commits
eae24ca
e304842
d3ad8f2
de37c65
555d5c7
e28ccf4
875da77
a62bde3
ba68404
5296c16
f14347f
51cab26
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This workflow file contains literal
JsRequireGhAw(...)fragments inside the JavaScriptrequire()calls, which will make the workflow fail at runtime. It looks like pkg/workflow/maintenance_workflow.go is emitting un-interpolated Go expressions; once that generator is fixed, please recompile/regenerate this workflow so therequire()calls resolve toprocess.env.GH_AW_HOME + '/actions/…'.See below for a potential fix: