diff options
author | Kamil Trzciński <ayufan@ayufan.eu> | 2015-10-05 17:42:50 +0000 |
---|---|---|
committer | Kamil Trzciński <ayufan@ayufan.eu> | 2015-10-05 17:42:50 +0000 |
commit | 3eef0e18e0d4561c320b4a5d9c17d7ae7aaa3d42 (patch) | |
tree | 4cc8a625cb1baebfbc7864ff8eb7746d99610e76 /config/routes.rb | |
parent | 97f7edf37a7c3c17d5038a6cb83d007aa026664c (diff) | |
parent | 97a11136d3e7d7ed57c7571d296d7b50617dce16 (diff) | |
download | gitlab-ce-3eef0e18e0d4561c320b4a5d9c17d7ae7aaa3d42.tar.gz |
Merge branch 'refactor-build-service' into 'master'
Refactor Ci::Commit and Ci::Build to have all builds for same :sha on single page
This makes Ci::Commit to have only :sha and simplifies routing to have only :sha in path. The :ref and :push_data is now parameter of Ci::Build.
All commit related data (git author, message and .gitlab-ci.yml) is read directly from repository.
All code related for creating builds is moved to CreateBuildsService.
Status deduction is rewritten to make if more efficient and easier to integrate with Commit Status API.
This is partially working, tests are not yet touched.
This slightly changes view of Commit:
![Screen_Shot_2015-10-02_at_15.21.47](https://gitlab.com/gitlab-org/gitlab-ce/uploads/ad3f1ccdcc87659ea437d8db6c5b9f94/Screen_Shot_2015-10-02_at_15.21.47.png)
@dzaporozhets What do you think?
See merge request !1502
Diffstat (limited to 'config/routes.rb')
-rw-r--r-- | config/routes.rb | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/config/routes.rb b/config/routes.rb index 7bda7ade817..a21889631ed 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -30,12 +30,10 @@ Gitlab::Application.routes.draw do resource :charts, only: [:show] - resources :refs, constraints: { ref_id: /.*/ }, only: [] do - resources :commits, only: [:show] do - member do - get :status - get :cancel - end + resources :commits, only: [:show] do + member do + get :status + get :cancel end end |