-
Notifications
You must be signed in to change notification settings - Fork 18
Update nipunn1313-mypy plugins to the latest #2105
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Update plugins to use bookworm based build image and distroless base image. Fixes #2104.
stefanvanburen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems reasonable — did we have a similar issue w/ other python plugins?
| # https://github.com/nipunn1313/mypy-protobuf/tree/v3.7.0#requirements-to-run-typecheckers-on-stubs-generated-by-mypy-protobuf | ||
| # > Earlier releases might work, but aren't tested. | ||
| - "types-protobuf >= 5.28" | ||
| - "protobuf >= 5.28.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated these deps based on the latest documented versions.
| WORKDIR /app | ||
| RUN python -mvenv /app | ||
| ADD /requirements.txt requirements.txt | ||
| RUN . ./bin/activate \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Had to switch source to . for the build to work. Switched this plugin to use distroless python.
I think I'd updated other ones already. This plugin was building on an alpine image and deploying w/ debian based one. |
Update plugins to use bookworm based build image and distroless base image.
Fixes #2104.