Skip to content

Add Docker docs and test-docker justfile target#306

Merged
blink1073 merged 1 commit into
Calysto:mainfrom
blink1073:add-docs-on-docker
Mar 22, 2026
Merged

Add Docker docs and test-docker justfile target#306
blink1073 merged 1 commit into
Calysto:mainfrom
blink1073:add-docs-on-docker

Conversation

@blink1073
Copy link
Copy Markdown
Contributor

References

N/A

Description

Adds documentation for the pre-built Docker image and a test-docker justfile target that validates the image after building.

Changes

  • Add Docker installation section to README.md with instructions for pulling and running the image, including a note to click the token URL to open JupyterLab
  • Add .dockerignore to exclude .git, .venv, and cache files from Docker builds
  • Add test-docker justfile target that runs docker-build then verifies the octave kernelspec is listed and the octave executable is available
  • Update the docker-build CI job to use just test-docker after the Buildx build (with GHA layer caching preserved via load: true)
  • Remove .python-version (Python version is managed by Poetry)

Backwards-incompatible changes

None

Testing

Ran just test-docker locally — both checks pass (kernelspec listed, octave --version succeeds).

AI usage

  • AI was used to generate or assist with this code
  • The human author has reviewed and understands all AI-generated contributions

Tools: Claude (claude-sonnet-4-6)

- Add Docker installation section to README with token URL instructions
- Add .dockerignore to exclude .git, .venv, and cache files from builds
- Add test-docker justfile target that builds the image and verifies
  the octave kernelspec and octave executable are available
- Update docker-build CI job to use just test-docker with Buildx caching
- Remove .python-version (managed by Poetry)
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.64%. Comparing base (234777e) to head (57971dc).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #306   +/-   ##
=======================================
  Coverage   91.64%   91.64%           
=======================================
  Files           6        6           
  Lines         431      431           
=======================================
  Hits          395      395           
  Misses         36       36           
Flag Coverage Δ
linux 91.41% <ø> (ø)
windows 91.64% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@blink1073 blink1073 merged commit 83ed949 into Calysto:main Mar 22, 2026
27 checks passed
@blink1073 blink1073 deleted the add-docs-on-docker branch March 22, 2026 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant