-
Notifications
You must be signed in to change notification settings - Fork 283
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
Add format agent copy and equality methods #6119
Conversation
for more information, see https://pre-commit.ci
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6119 +/- ##
==========================================
+ Coverage 89.77% 89.82% +0.04%
==========================================
Files 88 88
Lines 23026 23150 +124
Branches 5036 5043 +7
==========================================
+ Hits 20672 20794 +122
- Misses 1623 1624 +1
- Partials 731 732 +1 ☔ View full report in Codecov by Sentry. |
…nto add-format-agent-copy * 'add-format-agent-copy' of github.com:jrackham-mo/iris: [pre-commit.ci] auto fixes from pre-commit.com hooks
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 @jrackham-mo, code all looks good to me, just a minor change in the whastnew and should be good to go!
Co-authored-by: Elias <[email protected]>
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.
lgtm, thanks!
🚀 Pull Request
Description
Closes #6108.
Adds a copy method to
FormatAgent
, which creates a newFormatAgent
from the format specs. Also adds an equality method which compares the format specs.The use case for this is to temporarily modify the
FORMAT_AGENT
in a custom load, and then restore it so that we don't change standardiris.load
behaviour outside of the custom load.