Skip to content

Experiment update endpoint#190

Merged
smacker merged 2 commits into
src-d:masterfrom
smacker:exp_put
Mar 1, 2018
Merged

Experiment update endpoint#190
smacker merged 2 commits into
src-d:masterfrom
smacker:exp_put

Conversation

@smacker

@smacker smacker commented Mar 1, 2018

Copy link
Copy Markdown
Contributor

Based on #188

Implements endpoint for #160

@smacker smacker requested a review from bzz March 1, 2018 14:51
@smacker smacker mentioned this pull request Mar 1, 2018
5 tasks

@dpordomingo dpordomingo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

smacker added 2 commits March 1, 2018 16:55
Signed-off-by: Maxim Sukharev <maxim@sourced.tech>
Signed-off-by: Maxim Sukharev <maxim@sourced.tech>
@bzz

bzz commented Mar 1, 2018

Copy link
Copy Markdown
Contributor

May be that is something obvious, but from the description it is not very clear, in what use-case update of an experiment is going to happen.

@smacker

smacker commented Mar 1, 2018

Copy link
Copy Markdown
Contributor Author

@bzz as usual button edit that allows to change name or description.

@smacker smacker merged commit e952dd5 into src-d:master Mar 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants