Skip to content

Get CI working again - #313

Open
rofinn wants to merge 7 commits into
JuliaDatabases:masterfrom
rofinn:rf/fix-ci
Open

Get CI working again#313
rofinn wants to merge 7 commits into
JuliaDatabases:masterfrom
rofinn:rf/fix-ci

Conversation

@rofinn

@rofinn rofinn commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Noticed in #312 that the CI and tests are currently failing. This is just an attempt to get them green again.

Changes

  • Updated CI action versions
  • Updated codecov actions are commented out since they now require a token https://github.com/codecov/codecov-action#breaking-changes
  • CI matrix to only cover Julia LTS and latest, along with the oldest and newest supported Postgres versions
  • Bump macOS-13 x64 to macos-26-intel. This will likely be the last supported Intel Mac image
  • Pinned HTTP.jl to v1 for error code script given TLS issue on v2 HTTP v2 fails against postgresql.org (TLS 1.3 alert 40) JuliaWeb/HTTP.jl#1338
  • Updated allocation tests on Julia >= 1.11 due to the Julia Array/Memory changes
  • Bumped supported PostgreSQL versions
  • Updated the error code generation script to handle both deprecated and additional codes
  • Error code generation script now merges the codes from the oldest and newest supported Postgres versions
  • Added a dependabot.yml file to try and reduce the maintenance burden for this package.

Actions

  • Setup a codecov token and uncomment actions
  • Enable dependabot on this repo
  • Revert the HTTP pin if the TLS issue is resolve in time

@rofinn
rofinn marked this pull request as draft July 27, 2026 17:25
@rofinn rofinn changed the title [WIP] Get CI working again Get CI working again Jul 27, 2026
rofinn added 3 commits July 27, 2026 11:24
HTTP.jl 2.x's rewritten TLS backend (Reseau) fails the TLS handshake
against postgresql.org (TLS 1.3 alert 40), but the MbedTLS v1 still
works.

Opened an issue with HTTP for now: JuliaWeb/HTTP.jl#1338
Starting in Julia 1.11 the `Array` type now wraps `Memory`.
This improves performance on various functions, but the extra
indirection adds an allocation when calling `unsafe_wrap` which impacts
our text-formatted number tests.
1. Oldest supported postgres version is v14
2. Latest released version if v18
3. Reduces the size of the CI matrix to avoid testing on every postgres
   version supported.
4. Bumps the LTS Julia version to 1.10
5. Updated the error_code.jl generation script to address a few issues
   and merge codes from multiple releases (covering both additions and
   deprecations we still need to support).
@rofinn
rofinn marked this pull request as ready for review July 28, 2026 06:44
rofinn added 2 commits July 28, 2026 10:25
- This would at least help catch stale actions
- Doesn't catch old Julia or Postgres versions, but we're mostly using
  latest which helps
- I believe this requires setup by a repo / org owner
@rofinn rofinn mentioned this pull request Jul 28, 2026
- Removes pinning HTTP to v1 in the CI job
- Updates the HTTP.get call to also extract the .body as per the new
  API.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant