search

tinkerbell / actions

Suite of Tinkerbell Actions for use in Tinkerbell Workflows
Apache License 2.0
28 stars 43 forks source link

Fix CI for PRs from forks #61

Closed mmlb closed 3 years ago

mmlb commented 3 years ago

Description

Main work is to fix the Build and push new actions to Quay.io action to not attempt to login to Quay from PRs from forks. PRs from forks don't have access to secrets and thus the docker login step fails.

I've also cleaned up the workflow files a bit so that each step has a name and an empty space around it so the files are easier to grok.

I've also added a .envrc so we can make use of direnv + nix-shell.

Why is this needed

Should fix CI that is holding back #60.

mmlb commented 3 years ago

@gianarb or @thebsdbox, can you take a look/merge so we can get #60 over the hump.