Skip to content

Test and code improvements #19396

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
Merged
AbrilRBS merged 10 commits into from
Dec 18, 2025
Merged

Test and code improvements #19396

AbrilRBS merged 10 commits into from
Dec 18, 2025

Conversation

@memsharded
Copy link
Member

@memsharded memsharded commented Dec 17, 2025 •
edited
Loading

Changelog: Omit
Docs:Omit

Pure refactors and cleanings:

  • Format, pep8 and style
  • Unused imports
  • Unused variables
  • Removed old style (object) inheritance
  • Moved dead test file to running test file
  • Beautify tests with textwrap.dedent( for code snippets
  • Modernize exception testing to with pytest.raises(PermissionError):

Close #19397

@memsharded memsharded added this to the 2.25.0 milestone Dec 17, 2025
@memsharded memsharded mentioned this pull request Dec 17, 2025
Closed
1 task
@AbrilRBS AbrilRBS merged commit bd7ad6a into conan-io:develop2 Dec 18, 2025
16 checks passed
@AbrilRBS AbrilRBS deleted the test/improvements branch December 18, 2025 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AbrilRBS AbrilRBS AbrilRBS approved these changes

Assignees

@AbrilRBS AbrilRBS

Labels

None yet

Projects

None yet

Milestone

2.25.0

Development

Successfully merging this pull request may close these issues.

[question] Use of dynamic options definition in conan2.

2 participants

@memsharded @AbrilRBS