-
Notifications
You must be signed in to change notification settings - Fork 251
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 operational configuration to application configuration #98
Conversation
Thanks @technosophos |
c575ed6
to
5c796b0
Compare
I made a handful of small additional changes to try to disambiguate a few uses of the term "application". But I think there are probably more that we will need to make in future PRs. |
@technosophos |
I am working on it today. |
Per #95, this renames OperationalConfiguration (and all related terminology) to ApplicationConfiguration (with related terminology). BREAKING CHANGE. Closes #95
5c796b0
to
9d6877f
Compare
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
}, | ||
"labels": { | ||
"type":"object", | ||
"type": "object", | ||
"description": "A set of string key/value pairs used as arbitrary labels on this operational configuration.", |
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.
should probably be application configuration
?
}, | ||
"annotations": { | ||
"type":"object", | ||
"type": "object", | ||
"description": "A set of string key/value pairs used as arbitrary annotations on this operational configuration.", |
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.
should probably be application configuration
?
Per #95, this renames OperationalConfiguration (and all related terminology) to ApplicationConfiguration (with related terminology).
BREAKING CHANGE.
Closes #95