You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Verify that the conditional expressions for PROJECT_ID and CREDENTIALS_JSON use valid GitHub Actions expression syntax and yield the expected values for both staging and production environments.
The README introduces an input service_account_json_b64 while the action defines service_account_json. Ensure the documentation aligns with the actual composite action inputs.
service_account_json_b64: ${{ inputs.service_account_b64 }} # Must be base64 encoded
The action requires the service account JSON to be base64 encoded. How you provide this encoded value is up to your workflow design.
on:
- pull_request_target:+ pull_request:
types:
- opened
- edited
- synchronize
- reopened
Suggestion importance[1-10]: 9
__
Why: Using pull_request_target exposes secrets to untrusted PRs, so switching to pull_request enhances security during title linting.
High
Possible issue
Install gcloud and fix auth quoting
Remove the single quotes around the expression so the JSON is correctly injected. Also insert a step to install the GCloud SDK before authenticating to ensure the gcloud CLI is available for downstream commands.
-uses: 'google-github-actions/auth@v2'-with:- credentials_json: '${{ inputs.service_account_json }}'+- name: Set up GCloud SDK+ uses: google-github-actions/setup-gcloud@v1+- name: Authenticate with GCP service account key+ uses: google-github-actions/auth@v2+ with:+ credentials_json: ${{ inputs.service_account_json }}+
Suggestion importance[1-10]: 8
__
Why: Ensures the GCloud SDK is installed before authentication and fixes the quoting on credentials_json, enabling proper injection.
Medium
Fix README input name mismatch
The composite action still expects service_account_json, not service_account_json_b64. Align the README with the actual input name so users don’t get errors when they pass secrets.
-service_account_json_b64: ${{ inputs.service_account_b64 }} # Must be base64 encoded+service_account_json: ${{ inputs.service_account_b64 }} # Must be base64 encoded
Suggestion importance[1-10]: 5
__
Why: The README references a non-existent service_account_json_b64 input, leading to confusion; aligning it with service_account_json prevents user errors.
Low
General
Unquote credentials_json expression
Quoting the entire expression may prevent proper evaluation of the JSON. Remove the surrounding quotes so the expression expands correctly.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Type
Enhancement, Documentation
Description
Refactor deploy workflow to composite action
Add dynamic project and credentials env variables
Simplify PR lint workflow configuration
Update README and action inputs documentation
Changes walkthrough 📝
deploy-functions.yml
Refactor deploy workflow to composite action.github/workflows/deploy-functions.yml
PROJECT_IDandCREDENTIALS_JSONenvaction.yml
Update composite action auth and inputsaction.yml
environmentinput, requireproject_idgoogle-github-actions/auth@v2for auth steplint-pr-title.yml
Update PR lint workflow config.github/workflows/lint-pr-title.yml
pull_request_targeteventREADME.md
Update README usage and inputsREADME.md
service_account_json_b64