-
Notifications
You must be signed in to change notification settings - Fork 895
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
Add ABI test against PG snapshot #7457
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7457 +/- ##
==========================================
+ Coverage 80.06% 82.19% +2.12%
==========================================
Files 190 230 +40
Lines 37181 43252 +6071
Branches 9450 10875 +1425
==========================================
+ Hits 29770 35552 +5782
- Misses 2997 3380 +383
+ Partials 4414 4320 -94 ☔ View full report in Codecov by Sentry. |
de3209e
to
72ab951
Compare
745db2e
to
2d40581
Compare
The added workflow This is due to the warnings like the following being thrown, which break our tests:
|
2d40581
to
2cfecb8
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.
We are building pg twice in the workflow while technically we only need to build it once. But since this only runs nightly and it would probably complicate the workflow a bit I think this is fine.
e39dc0c
to
d439cab
Compare
This adds a test to check the ABI compatibility with PG snapshot. Timescaledb is built against the mininum abi version of postgres supported, for each major version of PG and tested against the latest snapshot.
d439cab
to
bcb9306
Compare
This adds a test to check the ABI compatibility with PG snapshot. Timescaledb is built against the mininum abi version of postgres supported, for each major version of PG and tested against the latest snapshot.
Disable-check: force-changelog-file