Skip to content

Conversation

@bcbrookman
Copy link
Contributor

@bcbrookman bcbrookman commented May 27, 2025

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
Member

@bcbrookman thank you for this fix.

@jmeridth jmeridth merged commit 1ccb13f into github: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