Skip to content

Make jupyter/Dockerfile build again#945

Merged
mhucka merged 6 commits intoquantumlib:mainfrom
mhucka:mh-fix-dockerfile-lint
Nov 10, 2025
Merged

Make jupyter/Dockerfile build again#945
mhucka merged 6 commits intoquantumlib:mainfrom
mhucka:mh-fix-dockerfile-lint

Conversation

@mhucka
Copy link
Copy Markdown
Collaborator

@mhucka mhucka commented Nov 7, 2025

jupyter/Dockerfile failed to build for me due to changes in more recent versions of Jupyter and Ubuntu. Downgrading some packages like notebook and using a different combination of commands for the notebook extensions resolved the failures.

In addition, this PR also addresses lint errors reported by hadolint.

This Dockerfile failed to build for me due to changes in more recent
versions of Jupyter and Ubuntu. These changes make it work, and also
address lint errors reported by `hadolint`.
@github-actions github-actions Bot added the size: S 10< lines changed <50 label Nov 7, 2025
Comment thread jupyter/Dockerfile Dismissed
Comment thread jupyter/Dockerfile Dismissed
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request successfully fixes the build for jupyter/Dockerfile by downgrading packages and restructuring the Dockerfile commands. The changes to pin package versions and optimize apt-get usage are excellent for reproducibility and efficiency. I've identified a couple of areas for further improvement to enhance layer caching and ensure fully reproducible builds, which I've detailed in the comments.

Comment thread jupyter/Dockerfile
Comment thread jupyter/Dockerfile Outdated
@mhucka mhucka added this pull request to the merge queue Nov 10, 2025
Merged via the queue into quantumlib:main with commit c1594b9 Nov 10, 2025
52 checks passed
@mhucka mhucka deleted the mh-fix-dockerfile-lint branch November 10, 2025 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants