-
Notifications
You must be signed in to change notification settings - Fork 2
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
Rename ProgramInfo.config_interface to interface_description #631
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
💯 Much clearer IMO.
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.
looks good but needs a changelog and potentially breaking with a spec change tag too since this does change the interface
Co-authored-by: peg <peg@magmacollective.org>
Co-authored-by: peg <peg@magmacollective.org>
Co-authored-by: peg <peg@magmacollective.org>
@JesseAbram i didn't see the spec tag, nice. i updated the changelog too |
0cb4823
to
222dbd3
Compare
222dbd3
to
f849dc2
Compare
@JesseAbram can you reapprove? |
Since the
config_interface
field will be used to describe the interfaces for both the config and for auxiliary data (and future additional and even user-defined interfaces), it makes sense to rename it tointerface_description
to reduce ambiguity.