Skip to content

fix: use opscontract.WriteOutput in set-config#107

Merged
ecPablo merged 1 commit into
mainfrom
ecpablo/use-write-contract-set-config
Jun 26, 2026
Merged

fix: use opscontract.WriteOutput in set-config#107
ecPablo merged 1 commit into
mainfrom
ecpablo/use-write-contract-set-config

Conversation

@ecPablo

@ecPablo ecPablo commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

We can get rid of some types in set-config operations and instead use the types provided by CLDF. This way we avoid having redundant types in the codebase and centralize evm common types with the types provided by opsgen pkgs.

@github-actions

Copy link
Copy Markdown

Release impact (release-please)

Current version 0.7.1 (on main)
After merge 0.7.2 (patch bump)

PR title: fix: use opscontract.WriteOutput in set-config

Merging this PR as-is will contribute a patch bump to the next release-please release PR.

Conventional commit → bump

Intent PR title prefix Bump
Bug fix fix: patch
New feature feat: minor
Breaking change feat!: / fix!: or BREAKING CHANGE: / BREAKING-CHANGE: in description major
No release chore:, docs:, ci:, refactor:, etc. none

Update the PR title before merge if you need a different bump (squash commit message = PR title).

Preview is based on this PR title only. The release-please release PR may include other unreleased commits already on main.

@cl-sonarqube-production

Copy link
Copy Markdown

@ecPablo ecPablo marked this pull request as ready for review June 26, 2026 05:21
@ecPablo ecPablo requested a review from a team as a code owner June 26, 2026 05:21
@ecPablo ecPablo requested a review from graham-chainlink June 26, 2026 05:36
@ecPablo ecPablo merged commit e1a60bb into main Jun 26, 2026
17 checks passed
@ecPablo ecPablo deleted the ecpablo/use-write-contract-set-config branch June 26, 2026 05:57
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.

2 participants