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

DS-1462 Upgrade Python to 3.12 #922

Merged
merged 6 commits into from
Sep 13, 2024
Merged

Conversation

JackPlowman
Copy link
Contributor

@JackPlowman JackPlowman commented Nov 14, 2023

Task Branch Pull Request

https://nhsd-jira.digital.nhs.uk/browse/DS-1462

Description of Changes

This PR upgrades Python to 3.12

@JackPlowman JackPlowman force-pushed the task/DS-1462_Upgrade_Python_Version branch 2 times, most recently from b732ea9 to 95ee939 Compare November 23, 2023 15:23
@JackPlowman JackPlowman force-pushed the task/DS-1462_Upgrade_Python_Version branch 2 times, most recently from 3696d48 to bb3d1fe Compare December 18, 2023 10:54
Copy link

sonarcloud bot commented Dec 18, 2023

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@mabe13 mabe13 force-pushed the task/DS-1462_Upgrade_Python_Version branch 2 times, most recently from c257ee5 to 32bef47 Compare August 8, 2024 19:16
Copy link

sonarcloud bot commented Aug 8, 2024

@mabe13 mabe13 force-pushed the task/DS-1462_Upgrade_Python_Version branch from 34c711a to 137f93a Compare September 12, 2024 15:22
@mabe13 mabe13 marked this pull request as ready for review September 12, 2024 15:48
@mabe13 mabe13 requested a review from a team as a code owner September 12, 2024 15:48
Copy link

sonarcloud bot commented Sep 12, 2024

Copy link
Collaborator

@Tomdango Tomdango left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy with the changes to the tests. Only query is removing the update of the packages inside the container - however, I'm happy to allow it as we should be pulling the latest lambda image and not messing with it too much.

I'm also assuming it broke some things in the change from Amazon Linux 2 to Amazon Linux 2023.

@mabe13 mabe13 merged commit cf19362 into develop Sep 13, 2024
20 checks passed
@mabe13 mabe13 deleted the task/DS-1462_Upgrade_Python_Version branch September 13, 2024 08:22
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.

3 participants