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.

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

deploying: Use the same version of go as specified in go.mod #841

Merged
merged 2 commits into from
Jun 20, 2023
Merged

deploying: Use the same version of go as specified in go.mod #841

merged 2 commits into from
Jun 20, 2023

Conversation

kachick
Copy link
Collaborator

@kachick kachick commented Jun 20, 2023

Same as #838, #840

@kachick kachick requested a review from pankona as a code owner June 20, 2023 12:04
@kachick
Copy link
Collaborator Author

kachick commented Jun 20, 2023

Looks not inputting firebaseServiceAccount 🤔

Run FirebaseExtended/action-hosting-deploy@v0
  with:
    entryPoint: hashira-web
    repoToken: ***
    projectId: hashira-web
    expires: 7d
    firebaseToolsVersion: latest
/home/runner/work/_actions/FirebaseExtended/action-hosting-deploy/v0/bin/action.min.js:[2](https://github.com/pankona/hashira/actions/runs/5322334209/jobs/9638638125?pr=841#step:6:2)[10](https://github.com/pankona/hashira/actions/runs/5322334209/jobs/9638638125?pr=841#step:6:11)2
        throw new Error(`Input required and not supplied: ${name}`);
        ^

Error: Input required and not supplied: firebaseServiceAccount

But having same setting with -merge workflow. My permission issue?

@kachick kachick merged commit 6ca5e76 into pankona:master Jun 20, 2023
@kachick kachick deleted the sync-gomod-deploy branch June 20, 2023 12:17
kachick added a commit that referenced this pull request Jun 20, 2023
Because contributor and dependebot does not have the rights to read secret

#841 (comment)
kachick added a commit that referenced this pull request Jun 20, 2023
Because contributor and dependebot does not have the rights to read secret

#841 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant