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.
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
#3234 exclude extra field when represent model #3241
#3234 exclude extra field when represent model #3241
Conversation
please review |
I thought it was right to exclude "extra field". Let me know if I'm thinking in a different direction from main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for this, good catch
please update. |
Co-authored-by: Samuel Colvin <samcolvin@gmail.com>
Co-authored-by: Samuel Colvin <samcolvin@gmail.com>
updated |
HI @COCOLMAN you seem to have prevented me from contributing to your PR which means I can't fix the small issue in tests. Please accept this change. In future it would be easier if you gave permissions to maintainers to contribute to your pull requests for situations exactly like this. |
Co-authored-by: Samuel Colvin <samcolvin@gmail.com>
Thx. I will check |
Thanks so much. |
Change Summary
has been modified to exclude unpredictable "extra" fields from "repr"
Related issue number
fix #3234
Checklist
changes/<pull request or issue id>-<github username>.md
file added describing change(see changes/README.md for details)