Skip to content

Clean up naming in image builder workflow#104

Merged
tillkruss merged 5 commits intocachewerk:mainfrom
szepeviktor:patch-1
Aug 1, 2023
Merged

Clean up naming in image builder workflow#104
tillkruss merged 5 commits intocachewerk:mainfrom
szepeviktor:patch-1

Conversation

@szepeviktor
Copy link
Copy Markdown
Contributor

@szepeviktor szepeviktor commented Aug 1, 2023

First I've started up with adding checks for Dockerfiles using Hadolint.
Then I've read the current docker.yml file and got mental dizziness 😵

This PR cleans up naming and a few problems in the workflow file.
Replaces ❌esoteric names with publicly understandable ones.
One of my sources: https://docs.docker.com/engine/reference/commandline/build/

@szepeviktor
Copy link
Copy Markdown
Contributor Author

How should I test it? I have no permission to start this workflow manually.

Comment thread .github/workflows/images.yml Outdated
@szepeviktor szepeviktor mentioned this pull request Aug 1, 2023
@tillkruss tillkruss merged commit 939e8a6 into cachewerk:main Aug 1, 2023
@szepeviktor szepeviktor deleted the patch-1 branch August 1, 2023 22:45
@szepeviktor
Copy link
Copy Markdown
Contributor Author

I think the naming is esoteric again.

@szepeviktor
Copy link
Copy Markdown
Contributor Author

szepeviktor commented Aug 1, 2023

The output looks like this project is not open-souce (not welcoming) but for internal use only.

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