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

test: docker build and docker run workflows #124

Closed
wants to merge 13 commits into from
Closed

test: docker build and docker run workflows #124

wants to merge 13 commits into from

Conversation

thesarfo
Copy link

Improve Docker build process and CI workflow

Referred Issue - #66

Description

I introduced several changes to optimize the Docker build process, enhance the CI workflow, and improve overall project consistency.

Changes made:

  1. Modified Dockerfile:

    • Optimized the build process for better consistency and reliability.
    • Updated dependency installation and pruning steps.
  2. Updated .github/workflows/ci.yml:

    • Added steps in the CI workflow to incorporate Docker build and run tests.
  3. Modified package.json:

    • Added new scripts related to Docker testing.
    • Updated dependencies or scripts as needed for the new Docker workflow.
  4. Added .dockerignore:

    • Created a new .dockerignore file to exclude unnecessary files from the Docker build context, optimizing build time and image size
  5. Added test-docker.sh:

    • Created a new shell script to automate Docker build and run tests.
Testing

I have tested these changes locally. The Docker image builds successfully, and the new test script runs without errors. The updated CI workflow has been verified to work as expected.

@thesarfo thesarfo changed the title Add tests for docker build and docker run test: docker build and docker run workflows Jun 24, 2024
@hsb1007 hsb1007 requested a review from rajdip-b June 25, 2024 12:20
@thesarfo thesarfo closed this by deleting the head repository Aug 29, 2024
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