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

feat: add a reusable workflow "check-secret" #138

Merged

Conversation

kenji-miyake
Copy link
Contributor

@kenji-miyake kenji-miyake commented Apr 24, 2022

Description

It is useful if we can check whether a secret is set.

Related: actions/runner#520 (comment)

This can be used for autowarefoundation/autoware.universe#319.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
fix
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
@kenji-miyake kenji-miyake force-pushed the add-check-app-secrets branch from 1e315ce to cfe7282 Compare April 24, 2022 14:43
@kenji-miyake
Copy link
Contributor Author

kenji-miyake commented Apr 24, 2022

@kenji-miyake kenji-miyake enabled auto-merge (squash) April 24, 2022 14:46
@kenji-miyake kenji-miyake disabled auto-merge April 24, 2022 14:47
@kenji-miyake kenji-miyake changed the title feat: add check-secret.yaml feat: add a reusable workflow "check-secret" Apr 24, 2022
Kenji Miyake added 2 commits April 24, 2022 23:52
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
@kenji-miyake kenji-miyake enabled auto-merge (squash) April 24, 2022 14:53
@kenji-miyake
Copy link
Contributor Author

Since the diff is a bit weird, it's good to see c02a9bf and c323f87 separately.

@kenji-miyake kenji-miyake merged commit 0bcc0a6 into autowarefoundation:main Apr 25, 2022
@kenji-miyake kenji-miyake deleted the add-check-app-secrets branch April 25, 2022 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants