-
Notifications
You must be signed in to change notification settings - Fork 90
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
update lambdas to Python 3.11 #4241
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4241 +/- ##
==========================================
- Coverage 39.71% 37.60% -2.12%
==========================================
Files 779 769 -10
Lines 35336 33993 -1343
Branches 5156 5156
==========================================
- Hits 14035 12784 -1251
+ Misses 20757 20665 -92
Partials 544 544
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
dfb7784
to
a8f651c
Compare
05e2302
to
06751a3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is so much cleaner
yeah probably. maybe in a clean PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Description
changes:
notes
molecule lambda is not really used, so I haven't tested it manually, maybe makes sense to remove it?removedTODO