-
Notifications
You must be signed in to change notification settings - Fork 120
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
chore: Fix compatibility tests #2685
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2685 +/- ##
=======================================
Coverage 70.52% 70.52%
=======================================
Files 96 96
Lines 6652 6654 +2
Branches 769 769
=======================================
+ Hits 4691 4693 +2
Misses 1885 1885
Partials 76 76
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
1cd18ab
to
9479c06
Compare
0b061de
to
912f506
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.
Just one last change thank you!!
@@ -0,0 +1 @@ | |||
16.20.0 |
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.
sweet!
912f506
to
5233024
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.
LGTM! Thanks so much for fixing this, @kaloster 🙏 ❤️ Just one last question regarding whether we should test "3.12" as well thanks again!
fb896c7
to
bd1d302
Compare
bd1d302
to
15ed541
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.
Just one comment left. Otherwise LGTM!! Thanks for delivering this miracle ✨ 🚀 ✨
15ed541
to
eca5d20
Compare
V COOL!! LGTM 🎉🎉🎉 Thank you thank you!! |
@kaloster I think you need to LGTM this PR to merge lol Sorry ahahah |
Changes
pupetteer
andjest-pupetteer
As for Python 3.12 - it requires updating flatbuffers in order to work which is a breaking change and doesn't work with the current dep tree - it will also require migration