-
Notifications
You must be signed in to change notification settings - Fork 883
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
Remove backward direction for ABI tests #7495
base: main
Are you sure you want to change the base?
Conversation
720a1ab
to
1a67a07
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7495 +/- ##
==========================================
+ Coverage 80.06% 82.15% +2.08%
==========================================
Files 190 230 +40
Lines 37181 43188 +6007
Branches 9450 10867 +1417
==========================================
+ Hits 29770 35480 +5710
- Misses 2997 3383 +386
+ Partials 4414 4325 -89 ☔ View full report in Codecov by Sentry. |
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.
Looks good to me.
@mkindahl can u please point to related CI failures? |
Sure. An example is our nightly runs the last few days: https://github.com/timescale/timescaledb/actions/runs/12038215739/job/33563401918 |
Versions of the extension cannot be expected to always work in the backwards direction so we cannot reliably test that builds against later versions of PostgreSQL (e.g., 17.3) work with earlier versions (e.g., 17.0) if new functions are introduced in the later version.
1a67a07
to
49f31fb
Compare
Versions of the extension cannot be expected to always work in the backwards direction so we cannot reliably test that builds against later versions of PostgreSQL (e.g., 17.3) work with earlier versions (e.g., 17.0) if new functions are introduced in the later version.
Disable-check: force-changelog-file