-
Notifications
You must be signed in to change notification settings - Fork 8
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
[WIP] replace "state name" with "workspace" #24
base: master
Are you sure you want to change the base?
Conversation
In |
In |
Hmmm... Actually, if we add a |
The more I read at the code, the more I think this is a huge change... But I still think we should do it. type Workspace struct {
LastModified time.Time `json:"last_modified"`
Name string `json:"name"`
// Keep Lock info
Locked bool `json:"locked"`
LockInfo LockInfo `json:"lock"`
States []*terraform.State
Plans []*terraform.Plan
} |
I added a new |
Will fix #18. I think we should clarify this before going further because it will probably break the API (at least the internal one).
I still have a lot of questions for you @cryptobioz and @raphink so I open this PR so that we can discuss about that in comments.