Skip to content

Commit

Permalink
Merge pull request #63 from ethereum-optimism/02-10-fix_allow_comma_s…
Browse files Browse the repository at this point in the history
…eparated_chain_names

fix: allow comma separated chain names
  • Loading branch information
jakim929 authored Feb 11, 2025
2 parents 0a9b424 + 53e9ba8 commit c3fb3df
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 2 deletions.
5 changes: 5 additions & 0 deletions .changeset/shy-mails-report.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@eth-optimism/super-cli": patch
---

fixed bug with chainIdentifierToChain and added back navigation
2 changes: 1 addition & 1 deletion packages/cli/src/actions/deployCreateXCreate2.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ export const zodDeployCreateXCreate2Params = z.object({
alias: 'cargs',
}),
)
.min(4)
.min(1)
.optional(),
salt: z.string().describe(
option({
Expand Down
9 changes: 8 additions & 1 deletion packages/cli/src/commands/deploy/create2.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,14 @@ const DeployCreate2CommandEntrypoint = ({
const parseResult = zodDeployCreateXCreate2Params.safeParse(commandOptions);

return parseResult.success ? (
<DeployCreate2Command options={parseResult.data} />
<DeployCreate2Command
options={{
...parseResult.data,
chains: parseResult.data.chains.flatMap(chain =>
chain.split(',').map(chain => chain.trim()),
),
}}
/>
) : (
<StatusMessage variant="error">
{
Expand Down

0 comments on commit c3fb3df

Please sign in to comment.