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

Makefile for templates/openshift #4103

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

schuellerf
Copy link
Contributor

I just wanted to check #4102 locally.
maybe that's helpful for someone

just for manual checks if the template syntax is fine
Copy link
Member

@croissanne croissanne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm I wonder if this makes sense to put in the makefile. The only reason we process them in the github workflow is so that kubelinter can do a very basic check. The processing of the template when actually deploying is something that is out of our hands. If we put it in the makefile, then i'd just consider it as a helper for the github workflow.

@schuellerf
Copy link
Contributor Author

We could use it from GitHub Workflow but I just wanted to call it so see if there are syntax problems locally (avoid GitHub Action problems before pushing - the usual problem)

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.

None yet

2 participants