Skip to content
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

Positive test of beb672f #1

Closed
wants to merge 3 commits into from

Conversation

benjaminedwardwebb
Copy link
Owner

@benjaminedwardwebb benjaminedwardwebb commented Nov 8, 2024

Positive test of beb672f

Fix the Dockerfile such that it's based on a rust image that meets the
minimum supported rust version (MSRV) defined in cargo (1.81).

Signed-off-by: Ben Webb <[email protected]>
Add CI step that checks that the vtn.Dockerfile successfully builds a
docker image on each committed change.

We opt to not publish the image at this point in time.

See the prior commit for some motivation.

Refs: 5c188e5
Signed-off-by: Ben Webb <[email protected]>
@benjaminedwardwebb benjaminedwardwebb force-pushed the benwebb/fix-dockerfile-msrv branch from 2bd93e1 to beb672f Compare November 8, 2024 16:15
@benjaminedwardwebb benjaminedwardwebb changed the title Fix Dockerfile to support MSRV 1.81 Positive test of beb672f Nov 8, 2024
@benjaminedwardwebb benjaminedwardwebb deleted the benwebb/fix-dockerfile-msrv branch November 11, 2024 15:02
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.

2 participants