-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Get rid of 'machine' related terms #14081
Comments
Compatibility will need to be handled. |
Issues go stale after Mark the issue as fresh with If this issue is safe to close now please do so. Moderators: Add |
Issues go stale after Mark the issue as fresh with If this issue is safe to close now please do so. Moderators: Add |
Issues go stale after Mark the issue as fresh with If this issue is safe to close now please do so. Moderators: Add |
Is your enhancement related to a problem? Please describe.
"Machine" has been deprecated and should not be used anywhere in Che anymore.
So we should review our code and use another terms for that.
For example, at the moment task plugin gets info about target container by API as attribute of command
machineName
and provides this one for machine exec. So at least for task-plugin, machine exec and for che objects changes are required.Although we display that field as
containerName
for user, we still use termsmachineName
andmachine
in our code. Would be nice to replace these ones with more appropriate terms.We decided not to do such refactoring before GA, but we should consider to do it after.
The text was updated successfully, but these errors were encountered: