Skip to content

Kl/remove-new-tests#1989

Closed
kelemeno wants to merge 1 commit intomatter-labs:draft-v31from
kelemeno:kl/remove-new-tests
Closed

Kl/remove-new-tests#1989
kelemeno wants to merge 1 commit intomatter-labs:draft-v31from
kelemeno:kl/remove-new-tests

Conversation

@kelemeno
Copy link
Contributor

What ❔

Why ❔

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.

@kelemeno kelemeno changed the base branch from main to draft-v31 January 22, 2026 17:33
@kelemeno kelemeno closed this Jan 22, 2026
@chatgpt-codex-connector
Copy link

💡 Codex Review

import {GatewayUpgrade, GatewayUpgradeEncodedInput} from "contracts/upgrades/GatewayUpgrade.sol";

P1 Badge Drop stale GatewayUpgrade import

This new upgrade script imports contracts/upgrades/GatewayUpgrade.sol, but that file is deleted in this commit, so forge script (or any compilation of this script) fails with a missing-file error. If GatewayUpgrade is no longer part of the flow, remove the import; otherwise update it to the new location so the upgrade script compiles.


import {ContractsConfig, DeployedAddresses, TokensConfig} from "../DeployUtils.s.sol";

P1 Badge Fix DeployUtils import path

The script still imports ../DeployUtils.s.sol, which was removed/moved (the remaining DeployUtils now lives under deploy-scripts/utils/deploy/DeployUtils.sol), so compiling or running this script will fail with a missing-file error. Update the import to the new path or restore the file.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant