Skip to content

Validate mat_version (and revision) are integers in register UI#19

Merged
bdpedigo merged 4 commits into
mainfrom
copilot/validate-mat-version-int
Apr 30, 2026
Merged

Validate mat_version (and revision) are integers in register UI#19
bdpedigo merged 4 commits into
mainfrom
copilot/validate-mat-version-int

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 30, 2026 •
edited
Loading

  • Validate mat_version is an integer in register UI (early error response)
  • Validate revision is an integer in register UI (early error response)
  • Add tests for non-integer mat_version and revision inputs
  • Run linting (ruff format + ruff check --fix): removed unused NAME_FORMAT_MESSAGE import in assets.py
  • All 8 UI register tests pass

Copilot AI linked an issue Apr 30, 2026 that may be closed by this pull request
validate that mat version is an int #16
Closed
Copilot AI and others added 2 commits April 30, 2026 20:12
Copilot AI changed the title [WIP] Fix validation for mat version input in register UI Validate mat_version (and revision) are integers in register UI Apr 30, 2026
Copilot AI requested a review from bdpedigo April 30, 2026 20:14
@bdpedigo bdpedigo marked this pull request as ready for review April 30, 2026 20:19
@bdpedigo
Copy link
Copy Markdown
Collaborator

bdpedigo commented Apr 30, 2026

@copilot you should run linting (just lint)

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 30, 2026

@copilot you should run linting (just lint)

Done. Ran ruff format and ruff check --fix — the only auto-fixable issue was an unused NAME_FORMAT_MESSAGE import in assets.py (removed in 03bd57f). The 2 remaining E402 errors in tests/test_name_check.py are pre-existing and unrelated to this PR.

Copilot AI requested a review from bdpedigo April 30, 2026 20:23
@bdpedigo bdpedigo merged commit a763cfa into main Apr 30, 2026
@bdpedigo bdpedigo deleted the copilot/validate-mat-version-int branch April 30, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bdpedigo bdpedigo Awaiting requested review from bdpedigo

Assignees

@bdpedigo bdpedigo

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

validate that mat version is an int

2 participants

@bdpedigo