Skip to content

Reject blank catalog tool names - #463

Open
dajiaohuang wants to merge 1 commit into
analysis-tools-dev:masterfrom
dajiaohuang:fix/reject-blank-tool-names
Open

dajiaohuang wants to merge 1 commit into
analysis-tools-dev:masterfrom
dajiaohuang:fix/reject-blank-tool-names

Conversation

@dajiaohuang

Copy link
Copy Markdown

Summary

  • Reject empty and whitespace-only tool names in the catalog renderer, matching the nonblank-name requirement in CONTRIBUTING.md.
  • Clarify the existing 50-byte UTF-8 limit in the validation error.
  • Add a regression test through the normal entry-validation path.

Validation

  • git diff --check passed.
  • Rust/Cargo are not installed in this environment, so the Rust test suite was not run.
  • CI was not queried or awaited.

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.

1 participant