Skip to content

Pyton 3.12#42

Merged
pwalczysko merged 27 commits intoome:masterfrom
pwalczysko:p312
Nov 5, 2025
Merged

Pyton 3.12#42
pwalczysko merged 27 commits intoome:masterfrom
pwalczysko:p312

Conversation

@pwalczysko
Copy link
Member

@pwalczysko pwalczysko commented Sep 12, 2025

Rebased on top of #41

Similarly to ome/ansible-role-omero-web#54 (Step 1 in the description of the PR), some packages need to be installed using the default Python 3.9 interpreter.

Also, adding the upgrade of the testing framework here, aka ome/ansible-role-omero-web#54.

cc @khaledk2 @jburel @sbesson

@pwalczysko pwalczysko requested a review from khaledk2 September 12, 2025 15:04
@pwalczysko
Copy link
Member Author

Removed the pointing to branches - ready for review @khaledk2 , thank you.

@khaledk2
Copy link
Contributor

khaledk2 commented Nov 4, 2025

Could you please ensure the following:

  • Use the Fully Qualified Collection Name (FQCN) consistently throughout the tasks.
  • Review all import and include statements to verify that tasks and roles are correctly referenced and properly structured.

@pwalczysko
Copy link
Member Author

pwalczysko commented Nov 4, 2025

Could you please ensure the following:

Use the Fully Qualified Collection Name (FQCN) consistently throughout the tasks.

Review all import and include statements to verify that tasks and roles are correctly referenced and properly structured.

Thank you @khaledk2 - fixed in 86d2b42 - sorry for the omission, this PR has a long history.

Co-authored-by: khaledk2 <khaledk2@hotmail.com>
@khaledk2
Copy link
Contributor

khaledk2 commented Nov 4, 2025

@pwalczysko
Copy link
Member Author

@pwalczysko pwalczysko requested a review from khaledk2 November 4, 2025 19:44
@pwalczysko pwalczysko merged commit 52edd9b into ome:master Nov 5, 2025
4 checks passed
@pwalczysko pwalczysko deleted the p312 branch November 5, 2025 09:34
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