Skip to content
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

Fix broken CI after PG12 removal #5893

Merged
merged 1 commit into from
Jul 25, 2023

Conversation

lkshminarayanan
Copy link
Contributor

@lkshminarayanan lkshminarayanan commented Jul 25, 2023

The commit cdea343 updated the gh_matrix_builder.py script but failed to import PG_LATEST variable into the script thus breaking the CI. Import that variable to fix the CI tests.

Disable-check: force-changelog-file

The commit cdea343 updated the gh_matrix_builder.py script but failed
to import PG_LATEST variable into the script thus breaking the CI.
Import that variable to fix the CI tests.
@lkshminarayanan lkshminarayanan marked this pull request as ready for review July 25, 2023 10:48
@github-actions
Copy link

@sb230132, @fabriziomello: please review this pull request.

Powered by pull-review

@codecov
Copy link

codecov bot commented Jul 25, 2023

Codecov Report

Merging #5893 (502cd9e) into main (eaa1206) will decrease coverage by 0.85%.
The diff coverage is 91.42%.

@@            Coverage Diff             @@
##             main    #5893      +/-   ##
==========================================
- Coverage   87.90%   87.06%   -0.85%     
==========================================
  Files         239      239              
  Lines       55792    55704      -88     
  Branches    12359    12329      -30     
==========================================
- Hits        49046    48498     -548     
- Misses       4852     4873      +21     
- Partials     1894     2333     +439     
Files Changed Coverage Δ
src/compat/compat.h 92.15% <ø> (+4.85%) ⬆️
.../constraint_aware_append/constraint_aware_append.c 84.51% <ø> (-1.77%) ⬇️
src/planner/expand_hypertable.c 89.03% <ø> (-0.32%) ⬇️
src/planner/planner.c 90.86% <ø> (-0.63%) ⬇️
src/process_utility.c 91.86% <ø> (-0.81%) ⬇️
src/utils.c 61.18% <ø> (-0.30%) ⬇️
src/utils.h 83.33% <ø> (ø)
tsl/src/bgw_policy/job.c 85.04% <ø> (-0.72%) ⬇️
tsl/src/continuous_aggs/common.c 90.06% <ø> (-0.68%) ⬇️
tsl/src/continuous_aggs/materialize.c 76.61% <ø> (-0.27%) ⬇️
... and 10 more

... and 82 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@lkshminarayanan lkshminarayanan enabled auto-merge (rebase) July 25, 2023 12:02
@lkshminarayanan lkshminarayanan merged commit 61c288e into timescale:main Jul 25, 2023
40 of 41 checks passed
@lkshminarayanan lkshminarayanan deleted the pg12-removal-fix-ci branch July 28, 2023 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants