Skip to content

common/CI: Fix py 3.9 & 3.10 deprecation warnings, format with ruff#9348

Merged
EbonJaeger merged 2 commits into
mainfrom
package-checks-warnings
Jun 19, 2026
Merged

common/CI: Fix py 3.9 & 3.10 deprecation warnings, format with ruff#9348
EbonJaeger merged 2 commits into
mainfrom
package-checks-warnings

Conversation

@joebonrichie

Copy link
Copy Markdown
Contributor

Summary

Format with ruff
Fix python 3.9 deprecation warnings (Optional, List)
Fix python 3.10 deprecation warnings (Union)
Fix some miscellaneous warnings e.g. override, final for classes

Test Plan

Use package checks for a few updates as normal

Checklist

  • Package was built and tested against unstable
  • This change could gainfully be listed in the weekly sync notes once merged
  • I agree to license this contribution and all my previous contributions under the licensing terms in LICENSE.md and have the power and authority to grant those licenses.

@github-project-automation github-project-automation Bot moved this to Triage in Solus Jun 19, 2026
@joebonrichie
joebonrichie force-pushed the package-checks-warnings branch 2 times, most recently from e2bdb50 to 8a95e06 Compare June 19, 2026 15:22
@joebonrichie
joebonrichie force-pushed the package-checks-warnings branch from 8a95e06 to 9a17af3 Compare June 19, 2026 15:28

@EbonJaeger EbonJaeger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks!

@EbonJaeger
EbonJaeger merged commit 751921c into main Jun 19, 2026
2 checks passed
@EbonJaeger
EbonJaeger deleted the package-checks-warnings branch June 19, 2026 18:14
@github-project-automation github-project-automation Bot moved this from Triage to Done in Solus Jun 19, 2026
EbonJaeger added a commit that referenced this pull request Jun 19, 2026
**Summary**
- Adds more expansive monitoring.yaml checks

Depends on #9348

**Test Plan**

Check all the format checks are working

**Checklist**

- [ ] Package was built and tested against unstable
- [ ] This change could gainfully be listed in the weekly sync notes
once merged <!-- Write an appropriate message in the Summary section,
then add the "Topic: Sync Notes" label -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants