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.

Sign up for GitHub

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

Jump to bottom

test: remove 312 from docker tests #1542

Merged
merged 1 commit into from
Jul 23, 2024
Merged

test: remove 312 from docker tests #1542

merged 1 commit into from
Jul 23, 2024

Conversation

gavin-aguiar
Copy link
Contributor

Description

The python 3.12 image is not ready yet. Removing it from the docker tests. Will add later for images are created.

Fixes #

PR information

  • The title of the PR is clear and informative.
  • There are a small number of commits, each of which has an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
  • If applicable, the PR references the bug/issue that it fixes in the description.
  • New Unit tests were added for the changes made and CI is passing.

Quality of Code and Contribution Guidelines

@hallvictoria hallvictoria changed the title Remove 312 from docker tests test: remove 312 from docker tests Jul 23, 2024
@hallvictoria hallvictoria merged commit bc0f40f into dev Jul 23, 2024
24 of 28 checks passed
@hallvictoria hallvictoria deleted the gaaguiar/remove_312_docker branch July 23, 2024 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hallvictoria hallvictoria hallvictoria approved these changes

@vrdmr vrdmr Awaiting requested review from vrdmr vrdmr is a code owner

@YunchuWang YunchuWang Awaiting requested review from YunchuWang YunchuWang is a code owner

@pdthummar pdthummar Awaiting requested review from pdthummar pdthummar is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gavin-aguiar @hallvictoria