`invalid slice name` error should clarify min valid length is 3

This error confused me a bit at first, why was it invalid? Eventually I noticed a length of 3 characters for a slice key was successful. Looked up the source and the restriction appears to be due to a regex pattern constraint.

Perhaps in this case the error message could better indicate the failure was due to a minimum expected length?

```console
$ chisel find --release /opt/chisel-release base-distroless
2024/09/11 04:48:18 Processing /opt/chisel-release release...
error: invalid slice name "cc" in slices/base-distroless.yaml
```

https://github.com/canonical/chisel/blob/ae52f841c4b6ac024077bff6d345a63e153fbc45/internal/setup/setup.go#L562-L566

https://github.com/canonical/chisel/blob/ae52f841c4b6ac024077bff6d345a63e153fbc45/internal/setup/setup.go#L283-L284

	for sliceName, yamlSlice := range yamlPkg.Slices {
	match := snameExp.FindStringSubmatch(sliceName)
	if match == nil {
	return nil, fmt.Errorf("invalid slice name %q in %s", sliceName, pkgPath)
	}

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

`invalid slice name` error should clarify min valid length is 3 #158

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

	// snameExp matches only the slice name, without the leading package name.
	var snameExp = regexp.MustCompile(`^([a-z](?:-?[a-z0-9]){2,})$`)

invalid slice name error should clarify min valid length is 3 #158

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

`invalid slice name` error should clarify min valid length is 3 #158