diff options
author | Kamil Trzcinski <ayufan@ayufan.eu> | 2016-03-09 18:24:02 +0300 |
---|---|---|
committer | Kamil Trzcinski <ayufan@ayufan.eu> | 2016-03-14 15:20:35 +0300 |
commit | 0672258915a0cf444802ffc50ad1cd914f4f11d4 (patch) | |
tree | f8e389d36a5eb9a761e5b283fb0e2adddda60f9c /lib | |
parent | 37ba5a12b515172b76d28e112ab9899823163717 (diff) |
Cleanup CiCommit and CiBuild
- Remove all view related methods from Ci::Build and CommitStatus
- Remove unused Ci::Commit and Ci::Build methods
- Use polymorphism to render different types of CommitStatus
Diffstat (limited to 'lib')
-rw-r--r-- | lib/api/entities.rb | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/lib/api/entities.rb b/lib/api/entities.rb index 5b5b8bd044b..de58ef176cd 100644 --- a/lib/api/entities.rb +++ b/lib/api/entities.rb @@ -401,13 +401,6 @@ module API expose :id, :status, :stage, :name, :ref, :tag, :coverage expose :created_at, :started_at, :finished_at expose :user, with: User - # TODO: download_url in Ci:Build model is an GitLab Web Interface URL, not API URL. We should think on some API - # for downloading of artifacts (see: https://gitlab.com/gitlab-org/gitlab-ce/issues/4255) - expose :download_url do |repo_obj, options| - if options[:user_can_download_artifacts] - repo_obj.artifacts_download_url - end - end expose :artifacts_file, using: BuildArtifactFile, if: -> (build, opts) { build.artifacts? } expose :commit, with: RepoCommit do |repo_obj, _options| if repo_obj.respond_to?(:commit) |