Skip to content

Conversation

@MridulS
Copy link
Member

@MridulS MridulS commented Jul 30, 2025

Mantid supports python 3.11 now https://github.com/mantidproject/mantid/releases

- cmake=4.0.0
- conan=1.66.0
- cppcheck=2.6.2
- cppcheck=2.18.0
Copy link
Member

Choose a reason for hiding this comment

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

Was this necessary for updating to py 3.11?

Copy link
Member Author

@MridulS MridulS Aug 4, 2025

Choose a reason for hiding this comment

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

yes, the min cppcheck that is built on conda-forge with py3.11 is 2.18.0 :/

Comment on lines +39 to +52
uninitMemberVar
passedByValue
duplInheritedMember
throwInNoexceptFunction
returnByReference
constParameterReference
constStatement
normalCheckLevelMaxBranches
constVariableReference
missingReturn
passedByValueCallback
unreadVariable
accessMoved
unusedStructMember
Copy link
Member Author

Choose a reason for hiding this comment

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

I don't know enough to comment if this is an actual issue or not.

Copy link
Member

Choose a reason for hiding this comment

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

Can you open an issue so we don't forget looking into this?

Copy link
Member Author

Choose a reason for hiding this comment

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

Created #3740 to keep a track, maybe we can merge this in for now to move with dropping python 3.10?

@MridulS MridulS merged commit 5e6c262 into scipp:main Aug 4, 2025
4 checks passed
SimonHeybrock added a commit that referenced this pull request Aug 5, 2025
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