-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
- name: Build and push Docker image | ||
uses: docker/build-push-action@v5 | ||
with: | ||
context: . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am curious if this is needed, see docs:
context | String | Build's context is the set of files located in the specified PATH or URL(default Git context) |
---|
context String Build's context is the set of files located in the specified PATH or URL (default Git context)
x-common-variables: &test-variables | ||
HOST: http://testapp | ||
COOKIE_NAME: _proxy | ||
TOKEN_SECRET: Rm9yIEdvZCBzbyBsb3ZlZCB0aGUgd29ybGQgdGhhdCBoZSBnYXZlIGhpcyBvbmUgYW5kIG9ubHkgU29uLCB0aGF0IHdob2V2ZXIgYmVsaWV2ZXMgaW4gaGltIHNoYWxsIG5vdCBwZXJpc2ggYnV0IGhhdmUgZXRlcm5hbCBsaWZlLiAtIEpvaG4gMzoxNg== |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm surprised this didn't trigger a warning that a secret was committed. I wonder if we can use something simpler here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure if this repo has all the settings set like other repos.
I may be able to use something simpler, but I think I ran into issues if I did something too simple like abc123
Added
Changed
Removed
Note: Github Actions will fail if there are any env_files, even if they are part of a service that is not being used by Github Actions.