Skip to content

Conversation

@forever8896
Copy link
Contributor

Makes the 'command' field optional in NewArgs, allowing users to run 'pop new' without subcommands to get a beautiful interactive selection with emojis and engaging descriptions.

  • 🌐 Parachain: Build your own blockchain with custom logic, token economics, and governance
  • 🧩 Pallet: Create reusable blockchain modules to add features like tokens, voting, or custom logic
  • 📝 Smart Contract: Write decentralized applications with ink! for the Polkadot ecosystem

Includes updated test patterns and display formatting for the optional command.

Resolves #575

image

Makes the 'command' field optional in NewArgs, allowing users to run
'pop new' without subcommands to get a beautiful interactive selection
with emojis and engaging descriptions.

- 🌐 Parachain: Build your own blockchain with custom logic, token economics, and governance
- 🧩 Pallet: Create reusable blockchain modules to add features like tokens, voting, or custom logic
- �� Smart Contract: Write decentralized applications with ink! for the Polkadot ecosystem

Includes updated test patterns and display formatting for the optional command.

Resolves r0gue-io#575
Copy link
Collaborator

@AlexD10S AlexD10S left a comment

Choose a reason for hiding this comment

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

Great to see you contributing here, thanks for helping improve Pop CLI!
To fix the CI, just run:

cargo +nightly fmt --all

That will automatically take care of the formatting.

I also left a few comments on the output messages to keep things consistent with the rest of the CLI.

@forever8896
Copy link
Contributor Author

It seems the coverage test got cancelled rather than failed, ran for a while. is that a fault in my PR I can fix? @AlexD10S

@AlexD10S
Copy link
Collaborator

It seems the coverage test got cancelled rather than failed, ran for a while. is that a fault in my PR I can fix? @AlexD10S

Yes, I saw it. I don't think it's related to this PR. I'll rerun it and check what's happening.

Copy link
Collaborator

@AlexD10S AlexD10S left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for fixing all the feedback.

@AlexD10S AlexD10S merged commit 0df981d into r0gue-io:main Jul 30, 2025
5 checks passed
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.

pop new: interactive

2 participants