Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add argprocessor unit test for removing URLs from pydantic error messages #582

Closed
cindyyuanjiang opened this issue Sep 21, 2023 · 0 comments · Fixed by #605
Closed

Add argprocessor unit test for removing URLs from pydantic error messages #582

cindyyuanjiang opened this issue Sep 21, 2023 · 0 comments · Fixed by #605
Assignees
@cindyyuanjiang
Labels
new-cli scope out future new-cli work for the next few months user_tools Scope the wrapper module running CSP, QualX, and reports (python)

Comments

@cindyyuanjiang
Copy link
Collaborator

PR #560 improves pydantic error messages by removing confusing URLs from these messages. We should add an argprocessor unit test to verify this.
@cindyyuanjiang cindyyuanjiang added feature request New feature or request ? - Needs Triage user_tools Scope the wrapper module running CSP, QualX, and reports (python) new-cli scope out future new-cli work for the next few months and removed feature request New feature or request labels Sep 21, 2023
@cindyyuanjiang cindyyuanjiang mentioned this issue Sep 21, 2023
Merged
@cindyyuanjiang cindyyuanjiang self-assigned this Sep 29, 2023
@cindyyuanjiang cindyyuanjiang mentioned this issue Sep 30, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cindyyuanjiang cindyyuanjiang

Labels
new-cli scope out future new-cli work for the next few months user_tools Scope the wrapper module running CSP, QualX, and reports (python)
Projects
None yet
Milestone
No milestone
2 participants
@mattahrens @cindyyuanjiang