Skip to content

build: DAL dependency update - #1724

Merged
patextreme merged 5 commits into
mainfrom
update/main/dal
Aug 5, 2026
Merged

build: DAL dependency update#1724
patextreme merged 5 commits into
mainfrom
update/main/dal

Conversation

@lfdt-bot

@lfdt-bot lfdt-bot commented Mar 1, 2026

Copy link
Copy Markdown
Contributor

About this PR

Updates:

Usage

Please merge!

I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.

If you have any feedback, just mention me in the comments below.

Configure Scala Steward for your repository with a .scala-steward.conf file.

Have a fantastic day writing Scala!

🔍 Files still referring to the old version numbers

The following files still refer to the old version numbers.
You might want to review and update them manually.

DEPENDENCIES.md
build.sbt
⚙ Adjust future updates

Add these to your .scala-steward.conf file to ignore future updates of these dependencies:

updates.ignore = [
  { groupId = "org.postgresql", artifactId = "postgresql" }
]

Or, add these to slow down future updates of these dependencies:

dependencyOverrides = [
  {
    pullRequests = { frequency = "30 days" },
    dependency = { groupId = "org.postgresql", artifactId = "postgresql" }
  }
]
labels: library-update, early-semver-patch, semver-spec-patch, early-semver-pre-release, semver-spec-pre-release, version-scheme:early-semver, old-version-remains, commit-count:1

Signed-off-by: Hyperledger Bot <hyperledger-bot@hyperledger.org>
@lfdt-bot
lfdt-bot requested a review from a team as a code owner March 1, 2026 00:34
@github-actions

github-actions Bot commented Mar 1, 2026

Copy link
Copy Markdown
Contributor

Integration Test Results (prism-node)

1 tests  ±0   1 ✅ ±0   4s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 46daaef. ± Comparison against base commit 7cb6649.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Mar 1, 2026

Copy link
Copy Markdown
Contributor

Integration Test Results

 23 files  ±0   23 suites  ±0   4s ⏱️ ±0s
 58 tests ±0   58 ✅ ±0  0 💤 ±0  0 ❌ ±0 
106 runs  ±0  106 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 46daaef. ± Comparison against base commit 7cb6649.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Mar 1, 2026

Copy link
Copy Markdown
Contributor

Unit Test Results

113 files  ±0  113 suites  ±0   1h 1m 13s ⏱️ - 2m 20s
927 tests ±0  919 ✅ ±0  8 💤 ±0  0 ❌ ±0 
934 runs  ±0  926 ✅ ±0  8 💤 ±0  0 ❌ ±0 

Results for commit 46daaef. ± Comparison against base commit 7cb6649.

♻️ This comment has been updated with latest results.

@yshyn-iohk
yshyn-iohk enabled auto-merge (squash) March 10, 2026 09:13
yshyn-iohk
yshyn-iohk previously approved these changes Mar 10, 2026
@sonarqubecloud

Copy link
Copy Markdown

@yshyn-iohk

Copy link
Copy Markdown
Contributor

Root cause: this dependency PR is very stale, the branch is behind current main, and the failing workflow runs are old enough that GitHub no longer serves the logs (HTTP 410), so there is no trustworthy current CI signal to auto-fix against.

Proposed action: recreate or manually rebase this upgrade against current main and re-run CI on a fresh branch before attempting merge.

Signed: Codex AI agent, posting on behalf of @yshyn-iohk

@sonarqubecloud

sonarqubecloud Bot commented Jun 8, 2026

Copy link
Copy Markdown

@yshyn-iohk

Copy link
Copy Markdown
Contributor

Confirming the earlier diagnosis: the only red check is Run integration tests (neoprism), which is the pre-existing keycloak-plugins image failure (identus-keycloak-plugins:0.2.0 -> manifest unknown) that was fixed on main in #1844 / #1845 — not caused by this Postgres JDBC patch bump (42.7.9 -> 42.7.10). Rebasing onto main should turn CI green, and it's good to merge after that.

Reviewed with Claude Code

@sonarqubecloud

sonarqubecloud Bot commented Aug 5, 2026

Copy link
Copy Markdown

@patextreme
patextreme disabled auto-merge August 5, 2026 07:59

@patextreme patextreme left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving on behalf of the team. All CI checks green, Claude review passed, and this is a patch-level Scala Steward dependency bump (postgresql 42.7.9 → 42.7.10).

@patextreme
patextreme merged commit 7b96ddc into main Aug 5, 2026
18 of 19 checks passed
@patextreme
patextreme deleted the update/main/dal branch August 5, 2026 08:03
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.

4 participants