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

Allow passing of different Dockerfile locations #93

Merged
merged 5 commits into from
Jan 25, 2023

Conversation

MarvinZeising
Copy link
Member

@MarvinZeising MarvinZeising commented Jan 25, 2023

Type of Change

  • Bugfix
  • Enhancement / new feature
  • Refactoring
  • Documentation

Description

The GitOps Action actually supports passing a docker-file argument. However, this action doesn't support it. By adding it here, it allows us to use this action even if we have a Dockerfile that isn't in the current working directory.

Checklist

  • Add relevant labels (for example type of change or patch/minor/major)
  • Make sure not to introduce some mistakes
  • Update documentation
  • Review the Contributing Guideline and sign CLA
  • Reference relevant issue(s) and close them after merging

@MarvinZeising MarvinZeising added the minor Pull requests with new features label Jan 25, 2023
@MarvinZeising MarvinZeising requested a review from a team as a code owner January 25, 2023 13:03
@github-actions
Copy link
Contributor

github-actions bot commented Jan 25, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@MarvinZeising
Copy link
Member Author

I have read the CLA Document and I hereby sign the CLA

@MarvinZeising
Copy link
Member Author

recheck

@@ -83,6 +83,8 @@ jobs:
docker-build-target: "any target"
# optional: name of the docker image, default: private/<repository_name>
docker-image: <your-image>
# optional: path to the Dockerfile, default: ./Dockerfile
Copy link
Contributor

Choose a reason for hiding this comment

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

💙

.github/workflows/template_gitops.yml Show resolved Hide resolved
@@ -74,13 +74,15 @@ on: [ push ]

jobs:
gitops:
uses: Staffbase/gha-workflows/.github/workflows/template_gitops.yml@v1.10.0
uses: Staffbase/gha-workflows/.github/workflows/template_gitops.yml@v1.11.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Could we create the release after the PR from Kai is also merged? 🤔

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure, if doesn't take days... 😅

@MarvinZeising MarvinZeising merged commit 4b17084 into main Jan 25, 2023
@MarvinZeising MarvinZeising deleted the MarvinZeising-patch-1 branch January 25, 2023 15:08
@github-actions github-actions bot locked and limited conversation to collaborators Jan 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
minor Pull requests with new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants