Update action to use prebuilt Docker image#140
Open
mrfelton wants to merge 11 commits intolaunchdarkly:mainfrom
Open
Update action to use prebuilt Docker image#140mrfelton wants to merge 11 commits intolaunchdarkly:mainfrom
mrfelton wants to merge 11 commits intolaunchdarkly:mainfrom
Conversation
Prebuild docker image
|
Bump |
blaqbern
reviewed
Aug 14, 2025
Contributor
blaqbern
left a comment
blaqbern
left a comment
There was a problem hiding this comment.
So sorry for the delay on this. Left one question for you, but otherwise looks great! We appreciate the contribution!
action.yml
Outdated
| runs: | ||
| using: 'docker' | ||
| image: 'Dockerfile' | ||
| image: 'docker://ghcr.io/ln-zap/find-code-references-in-pull-request:latest' |
Contributor
There was a problem hiding this comment.
Not sure we want to push to the ln-zap namespace. Can we use the launchdarkly organization here instead?
Merge changes from upstream
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #138
This pull request introduces several new workflows and updates to the existing workflows to enhance the continuous integration and delivery process. The changes include adding a continuous delivery workflow, a version check workflow, and updates to the main workflow for local testing.
New Workflows:
.github/workflows/cd.yml: Added a continuous delivery workflow to create releases and publish container images to the GitHub Container Registry when a PR is merged or the workflow is run manually..github/workflows/version-check.yml: Introduced a version check workflow to ensure the container image version has been incremented in the manifest file before merging a PR.Workflow Updates:
.github/workflows/main.yml: Updated the main workflow to include a local Docker registry for testing, setup Docker BuildX, and build and push the container image for local testing.Other Changes:
.version: Added a version file with the initial versionv1.0.0.action.yml: Updated the Docker image reference to use the GitHub Container Registry.