Skip to content

fix: add missing Python modules to Dockerfile#324

Merged
jmeridth merged 1 commit into
github-community-projects:mainfrom
bcbrookman:bugfix/dockerfile-missing-python-modules
May 27, 2025
Merged

fix: add missing Python modules to Dockerfile#324
jmeridth merged 1 commit into
github-community-projects:mainfrom
bcbrookman:bugfix/dockerfile-missing-python-modules

Conversation

@bcbrookman

@bcbrookman bcbrookman commented May 27, 2025

Copy link
Copy Markdown
Contributor

Pull Request

Proposed Changes

This change includes env.py and auth.py in the container image. These modules are currently missing, resulting in ModuleNotFoundError.

Note: This likely hasn't been reported before because action.yml still specifies the stale_repos:v1 image, so even when using later versions of this action, the image that's pulled is still stale_repos:v1. I also opened #325 to address that seperately.

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request
  • run make lint and fix any issues that you have introduced
  • run make test and ensure you have test coverage for the lines you are introducing
  • If publishing new data to the public (scorecards, security scan results, code quality results, live dashboards, etc.), please request review from @jeffrey-luszcz

Reviewer

  • Label as either fix, documentation, enhancement, infrastructure, maintenance or breaking

@jmeridth

Copy link
Copy Markdown
Collaborator

@bcbrookman thank you for this fix.

@jmeridth jmeridth merged commit 1ccb13f into github-community-projects:main May 27, 2025
9 checks passed
@bcbrookman bcbrookman deleted the bugfix/dockerfile-missing-python-modules branch May 27, 2025 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants