-
-
Notifications
You must be signed in to change notification settings - Fork 249
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
pg17beta2 support #1792
pg17beta2 support #1792
Conversation
interesting. Those tests also fail for me locally from They all have the same failure pattern:
I'm running |
YES! I was missing the |
tests pass locally now on both linux and MacOS. A+++ I'm doing the CI dance now to get it to install/use the 17 beta deb packages. |
As far as I can tell, this is ready to merge |
@eeeebbbbrrrr This needs a rebase. |
Some tests fail locally. Not sure what'll happen on CI, which is why I'm PRing this now: ``` test tests::proptests::tests::pg_date_literal_spi_roundtrip ... FAILED test tests::proptests::tests::pg_timetz_literal_spi_roundtrip ... FAILED test tests::proptests::tests::pg_timestamp_literal_spi_roundtrip ... FAILED test tests::proptests::tests::pg_time_literal_spi_roundtrip ... FAILED test tests::proptests::tests::pg_date_spi_roundtrip ... FAILED test tests::proptests::tests::pg_timetz_spi_roundtrip ... FAILED test tests::proptests::tests::pg_time_spi_roundtrip ... FAILED test tests::proptests::tests::pg_timestamp_spi_roundtrip ... FAILED ```
done |
oh nice. |
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.
thank you!
Starting on pg17beta2 support...
Some tests fail locally. I dunno if these passed before -- I didn't run the test suite first. I assume they didn't. Will investigate.
Not sure what'll happen on CI, which is why I'm PRing this now: