diff options
author | Katarzyna Kobierska <kkobierska@gmail.com> | 2016-08-30 13:03:29 +0200 |
---|---|---|
committer | Katarzyna Kobierska <kkobierska@gmail.com> | 2016-09-07 12:10:49 +0200 |
commit | 1a7a900d8e4a4a3a65b33054168fdd86f252b757 (patch) | |
tree | 8420740a7ddbbdcce6e18141684a2e8cbc326a49 /lib/api | |
parent | 2c8b830fdbf749e8bb7461d5c3ce4699b77ce3ca (diff) | |
download | gitlab-ce-1a7a900d8e4a4a3a65b33054168fdd86f252b757.tar.gz |
Improve code
Diffstat (limited to 'lib/api')
-rw-r--r-- | lib/api/lint.rb | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/lib/api/lint.rb b/lib/api/lint.rb index b1c6f52bccb..98010cf3b68 100644 --- a/lib/api/lint.rb +++ b/lib/api/lint.rb @@ -5,22 +5,18 @@ module API requires :content, type: String, desc: 'Content of .gitlab-ci.yml' end - post 'ci/lint' do - error = Ci::GitlabCiYamlProcessor.validation_message(params[:content]) - response = { - status: '', - error: '' - } + namespace 'ci' do + post '/lint' do + errors = Ci::GitlabCiYamlProcessor.validation_message(params[:content]) - if error.blank? - response[:status] = 'valid' - else - response[:error] = error - response[:status] = 'invalid' - end + status 200 - status 200 - response + if errors.blank? + { status: 'valid', errors: [] } + else + { status: 'invalid', errors: [errors] } + end + end end end end |