Skip to content

Commit

Permalink
Update new.rs (MystenLabs#17828)
Browse files Browse the repository at this point in the history
Edited for clarity.

## Description 

Saw in a merged PR. The use of "please" is not advised as it suggests
package naming rules are optional.

## Test plan 

How did you test the new or updated feature?

---

## Release notes

Check each box that your changes affect. If none of the boxes relate to
your changes, release notes aren't required.

For each box you select, include information after the relevant heading
that describes the impact of your changes that a user might notice and
any actions they must take to implement updates.

- [ ] Protocol: 
- [ ] Nodes (Validators and Full nodes): 
- [ ] Indexer: 
- [ ] JSON-RPC: 
- [ ] GraphQL: 
- [ ] CLI: 
- [ ] Rust SDK:
  • Loading branch information
ronny-mysten authored May 20, 2024
1 parent 3d11ab6 commit e0068d9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions external-crates/move/crates/move-cli/src/base/new.rs
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,8 @@ impl New {

if !Identifier::is_valid(&name) {
return Err(anyhow!(
"Invalid package name, please use lowercase letters, numbers, and underscores, \
should start with a lowercase letter"
"Invalid package name. Package name must start with a lowercase letter \
and consist only of lowercase letters, numbers, and underscores."
));
}

Expand Down

0 comments on commit e0068d9

Please sign in to comment.