Skip to content

Conversation

@GitHK
Copy link
Collaborator

@GitHK GitHK commented Jan 7, 2026

@GitHK GitHK self-assigned this Jan 7, 2026
@GitHK GitHK merged commit f2dd73d into main Jan 7, 2026
1 check passed
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a missing pipreqs dependency issue by ensuring pipreqs is installed during the Docker build and modifying the requirements generation logic to write to a temporary location instead of the user's code directory.

  • Installs pipreqs in the Docker image build process
  • Changes the generated requirements.txt path from the code directory to /tmp to avoid permission issues
  • Bumps version from 1.1.1 to 1.1.2 across all relevant configuration files

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
common/Dockerfile Adds pipreqs installation and reorders PATH environment variable
common/main.py Updates requirements generation to use /tmp path and improves variable naming
Makefile Updates TAG version variables to 1.1.2
.osparc/osparc-python-runner-tensorflow/metadata.yml Updates version to 1.1.2
.osparc/osparc-python-runner-pytorch/metadata.yml Updates version to 1.1.2
.bumpversion-tensorflow.cfg Updates current_version to 1.1.2
.bumpversion-pytorch.cfg Updates current_version to 1.1.2

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@GitHK GitHK deleted the install-permissions branch January 7, 2026 16:01
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.

Minor bug with pipreqs

2 participants