Fixing var name
This commit is contained in:
@@ -18,7 +18,7 @@ on:
|
||||
required: true
|
||||
type: "string"
|
||||
|
||||
pull_request_html:
|
||||
pull_request_html_url:
|
||||
description: "html url of the pull request being asked about"
|
||||
required: true
|
||||
type: "string"
|
||||
@@ -34,16 +34,7 @@ jobs:
|
||||
COMMENT_BODY: ${{ inputs.comment_body }}
|
||||
ISSUE_NUMBER: ${{ inputs.issue_number }}
|
||||
COMMENT_ID: ${{ inputs.comment_id }}
|
||||
PULL_REQUEST_HTML_URL: ${{ inputs.pull_request_html }}
|
||||
# (env var name kept as PULL_REQUEST_HTML_URL for clarity in the
|
||||
# run: script below - this maps from the `pull_request_html`
|
||||
# input, which is what both pr-ask-ive.yml and
|
||||
# pr-ask-reply-ive.yml actually pass via `with:`. A prior version
|
||||
# of this file declared the input as `pull_request_html_url` but
|
||||
# referenced `inputs.pull_request_html` here - neither name
|
||||
# matched the caller's `with:` key, so PULL_REQUEST_HTML_URL was
|
||||
# always empty and --pr-url="" made every /ask call fail before
|
||||
# ever reaching the LLM.
|
||||
PULL_REQUEST_HTML_URL: ${{ inputs.pull_request_html_url }}
|
||||
run: |
|
||||
question="${COMMENT_BODY#/ask}"
|
||||
question="$(echo "$question" | sed -e 's/^[[:space:]]*//')"
|
||||
|
||||
Reference in New Issue
Block a user