Skip to content

fix runtime upgrade call for governance upgrade script#517

Merged
gilescope merged 12 commits into
mainfrom
fix/ephemeral-env-runtime-upgrade-call
Feb 23, 2026
Merged

fix runtime upgrade call for governance upgrade script#517
gilescope merged 12 commits into
mainfrom
fix/ephemeral-env-runtime-upgrade-call

Conversation

@justinfrevert

@justinfrevert justinfrevert commented Jan 22, 2026

Copy link
Copy Markdown
Contributor

Overview

🗹 TODO before merging

  • Ready

📌 Submission Checklist

  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason:
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • Updated AGENTS.md if build commands, architecture, or workflows changed
  • No new todos introduced

🧪 Testing Evidence

Please describe any additional testing aside from CI:

  • Additional tests are provided (if possible)

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other:
  • N/A

Links

@github-actions

github-actions Bot commented Jan 22, 2026

Copy link
Copy Markdown
Contributor

kics-logo

KICS version: v2.1.16

Category Results
CRITICAL CRITICAL 0
HIGH HIGH 0
MEDIUM MEDIUM 96
LOW LOW 12
INFO INFO 83
TRACE TRACE 0
TOTAL TOTAL 191
Metric Values
Files scanned placeholder 31
Files parsed placeholder 31
Files failed to scan placeholder 0
Total executed queries placeholder 73
Queries failed to execute placeholder 0
Execution time placeholder 9

@justinfrevert justinfrevert marked this pull request as ready for review January 22, 2026 17:47
@justinfrevert justinfrevert requested a review from a team as a code owner January 22, 2026 17:47
Comment thread changes/changed/fix-gov-update-command.md Outdated
@gilescope gilescope enabled auto-merge February 5, 2026 05:10
@CLAassistant

CLAassistant commented Feb 5, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

Comment thread local-environment/src/commands/federatedRuntimeUpgrade.ts Outdated

@jacek-kurkowski-shielded jacek-kurkowski-shielded 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.

LGTM, tested locally with HF 7 -> 8 👍

@gilescope gilescope added this pull request to the merge queue Feb 23, 2026
Merged via the queue into main with commit 5f96be1 Feb 23, 2026
40 checks passed
@gilescope gilescope deleted the fix/ephemeral-env-runtime-upgrade-call branch February 23, 2026 16:33
gilescope pushed a commit that referenced this pull request Apr 8, 2026
Signed-off-by: Christos Palaskas <chris.palaskas@gmail.com>
Co-authored-by: Krisztian Pinter <krisztian.pinter@iohk.io>
m2ux added a commit that referenced this pull request Apr 23, 2026
Signed-off-by: Christos Palaskas <chris.palaskas@gmail.com>
Co-authored-by: Krisztian Pinter <krisztian.pinter@iohk.io>
Signed-off-by: Mike Clay <mike.clay@shielded.io>
m2ux added a commit that referenced this pull request Apr 23, 2026
Signed-off-by: Christos Palaskas <chris.palaskas@gmail.com>
Co-authored-by: Krisztian Pinter <krisztian.pinter@iohk.io>
Signed-off-by: Mike Clay <mike.clay@shielded.io>
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