From 126c4d995202b754634430cdaaea784f9c6dff4a Mon Sep 17 00:00:00 2001 From: Achilleas Pipinellis Date: Sat, 21 Oct 2017 21:05:32 +0300 Subject: Fix wording on CI disposable variables --- doc/ci/variables/README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/ci/variables/README.md b/doc/ci/variables/README.md index 73568757aaa..bdd416b8372 100644 --- a/doc/ci/variables/README.md +++ b/doc/ci/variables/README.md @@ -43,7 +43,7 @@ future GitLab releases.** | **CI_COMMIT_TAG** | 9.0 | 0.5 | The commit tag name. Present only when building tags. | | **CI_CONFIG_PATH** | 9.4 | 0.5 | The path to CI config file. Defaults to `.gitlab-ci.yml` | | **CI_DEBUG_TRACE** | all | 1.7 | Whether [debug tracing](#debug-tracing) is enabled | -| **CI_DISPOSABLE_ENVIRONMENT** | all | 10.1 | Mark that job is executed in a disposable environment (something that is created only for this job and disposed of/destroyed after the execution - all executors except `shell` and `ssh`). If the environment is disposable, it is set to true, otherwise it is not defined at all. | +| **CI_DISPOSABLE_ENVIRONMENT** | all | 10.1 | Marks that the job is executed in a disposable environment (something that is created only for this job and disposed of/destroyed after the execution - all executors except `shell` and `ssh`). If the environment is disposable, it is set to true, otherwise it is not defined at all. | | **CI_ENVIRONMENT_NAME** | 8.15 | all | The name of the environment for this job | | **CI_ENVIRONMENT_SLUG** | 8.15 | all | A simplified version of the environment name, suitable for inclusion in DNS, URLs, Kubernetes labels, etc. | | **CI_ENVIRONMENT_URL** | 9.3 | all | The URL of the environment for this job | @@ -74,7 +74,7 @@ future GitLab releases.** | **CI_SERVER_NAME** | all | all | The name of CI server that is used to coordinate jobs | | **CI_SERVER_REVISION** | all | all | GitLab revision that is used to schedule jobs | | **CI_SERVER_VERSION** | all | all | GitLab version that is used to schedule jobs | -| **CI_SHARED_ENVIRONMENT** | all | 10.1 | Mark that job is executed in a shared environment (something that is persisted across CI invocations like `shell` or `ssh` executor). If the environment is shared, it is set to true, otherwise it is not defined at all. | +| **CI_SHARED_ENVIRONMENT** | all | 10.1 | Marks that the job is executed in a shared environment (something that is persisted across CI invocations like `shell` or `ssh` executor). If the environment is shared, it is set to true, otherwise it is not defined at all. | | **ARTIFACT_DOWNLOAD_ATTEMPTS** | 8.15 | 1.9 | Number of attempts to download artifacts running a job | | **GET_SOURCES_ATTEMPTS** | 8.15 | 1.9 | Number of attempts to fetch sources running a job | | **GITLAB_CI** | all | all | Mark that job is executed in GitLab CI environment | -- cgit v1.2.3 From 0bfb2aff7c2d74428321b42a25f8b7e92b8407e0 Mon Sep 17 00:00:00 2001 From: Phil Hughes Date: Tue, 24 Oct 2017 17:41:19 +0100 Subject: Fixed user profile tab being off-screen --- app/assets/stylesheets/framework/blocks.scss | 9 ++++++++- changelogs/unreleased/fix-user-tab-activity-mobile.yml | 5 +++++ 2 files changed, 13 insertions(+), 1 deletion(-) create mode 100644 changelogs/unreleased/fix-user-tab-activity-mobile.yml diff --git a/app/assets/stylesheets/framework/blocks.scss b/app/assets/stylesheets/framework/blocks.scss index dbd990f84c1..8819a0c20f4 100644 --- a/app/assets/stylesheets/framework/blocks.scss +++ b/app/assets/stylesheets/framework/blocks.scss @@ -209,7 +209,6 @@ padding: 24px 0 0; .nav-links { - justify-content: center; width: 100%; float: none; @@ -217,6 +216,14 @@ float: none; } } + + li:first-child { + margin-left: auto; + } + + li:last-child { + margin-right: auto; + } } .group-info { diff --git a/changelogs/unreleased/fix-user-tab-activity-mobile.yml b/changelogs/unreleased/fix-user-tab-activity-mobile.yml new file mode 100644 index 00000000000..a7e4fcb4355 --- /dev/null +++ b/changelogs/unreleased/fix-user-tab-activity-mobile.yml @@ -0,0 +1,5 @@ +--- +title: Fixed user profile activity tab being off-screen on mobile +merge_request: +author: +type: fixed -- cgit v1.2.3 From 4598c00a8f565a33c4460ae920ce7a65ba6540cc Mon Sep 17 00:00:00 2001 From: Richard Clamp Date: Wed, 25 Oct 2017 06:50:26 +0300 Subject: Add spec for QA::Scenario::Entrypoint For added confidence, and because I plan to fiddle with some behaviours shortly, add spec testing to the newly extracted QA::Scenario::Entrypoint class. --- qa/spec/scenario/entrypoint_spec.rb | 46 +++++++++++++++++++++++++++++++++++++ 1 file changed, 46 insertions(+) create mode 100644 qa/spec/scenario/entrypoint_spec.rb diff --git a/qa/spec/scenario/entrypoint_spec.rb b/qa/spec/scenario/entrypoint_spec.rb new file mode 100644 index 00000000000..3fd068b641c --- /dev/null +++ b/qa/spec/scenario/entrypoint_spec.rb @@ -0,0 +1,46 @@ +describe QA::Scenario::Entrypoint do + subject do + Class.new(QA::Scenario::Entrypoint) do + tags :rspec + end + end + + context '#perform' do + let(:config) { spy('Specs::Config') } + let(:release) { spy('Runtime::Release') } + let(:runner) { spy('Specs::Runner') } + + before do + allow(config).to receive(:perform) { |&block| block.call config } + allow(runner).to receive(:perform) { |&block| block.call runner } + + stub_const('QA::Specs::Config', config) + stub_const('QA::Runtime::Release', release) + stub_const('QA::Specs::Runner', runner) + end + + it 'should set address' do + subject.perform("hello") + + expect(config).to have_received(:address=).with("hello") + end + + context 'no paths' do + it 'should call runner with default arguments' do + subject.perform("test") + + expect(runner).to have_received(:rspec) + .with(hash_including(files: 'qa/specs/features')) + end + end + + context 'specifying paths' do + it 'should call runner with paths' do + subject.perform('test', 'path1', 'path2') + + expect(runner).to have_received(:rspec) + .with(hash_including(files: %w(path1 path2))) + end + end + end +end -- cgit v1.2.3 From 05728e785ce7cd39c4c517aa0ea50b3bba44d537 Mon Sep 17 00:00:00 2001 From: Phil Hughes Date: Thu, 26 Oct 2017 11:14:31 +0100 Subject: [WIP] Move multi-file editor store to Vuex --- app/assets/javascripts/api.js | 1 + .../repo/components/new_branch_form.vue | 49 +++--- app/assets/javascripts/repo/components/repo.vue | 78 +++------- .../repo/components/repo_commit_section.vue | 167 ++++++++------------- .../repo/components/repo_edit_button.vue | 75 ++++----- .../javascripts/repo/components/repo_editor.vue | 147 +++++++----------- .../javascripts/repo/components/repo_file.vue | 19 +-- .../repo/components/repo_file_buttons.vue | 45 ++---- .../repo/components/repo_loading_file.vue | 10 +- .../repo/components/repo_prev_directory.vue | 28 ++-- .../javascripts/repo/components/repo_preview.vue | 33 ++-- .../javascripts/repo/components/repo_sidebar.vue | 116 +++----------- .../javascripts/repo/components/repo_tab.vue | 24 ++- .../javascripts/repo/components/repo_tabs.vue | 12 +- .../repo/helpers/monaco_loader_helper.js | 4 - app/assets/javascripts/repo/index.js | 82 +++++----- app/assets/javascripts/repo/mixins/repo_mixin.js | 17 --- app/assets/javascripts/repo/services/index.js | 28 ++++ app/assets/javascripts/repo/stores/actions.js | 94 ++++++++++++ .../javascripts/repo/stores/actions/branch.js | 20 +++ app/assets/javascripts/repo/stores/actions/file.js | 50 ++++++ app/assets/javascripts/repo/stores/actions/tree.js | 70 +++++++++ app/assets/javascripts/repo/stores/getters.js | 39 +++++ app/assets/javascripts/repo/stores/index.js | 15 ++ .../javascripts/repo/stores/mutation_types.js | 26 ++++ app/assets/javascripts/repo/stores/mutations.js | 54 +++++++ .../javascripts/repo/stores/mutations/branch.js | 9 ++ .../javascripts/repo/stores/mutations/file.js | 50 ++++++ .../javascripts/repo/stores/mutations/tree.js | 27 ++++ app/assets/javascripts/repo/stores/repo_store.js | 7 - app/assets/javascripts/repo/stores/state.js | 21 +++ app/assets/javascripts/repo/stores/utils.js | 73 +++++++++ app/views/shared/repo/_repo.html.haml | 4 +- 33 files changed, 910 insertions(+), 584 deletions(-) delete mode 100644 app/assets/javascripts/repo/mixins/repo_mixin.js create mode 100644 app/assets/javascripts/repo/services/index.js create mode 100644 app/assets/javascripts/repo/stores/actions.js create mode 100644 app/assets/javascripts/repo/stores/actions/branch.js create mode 100644 app/assets/javascripts/repo/stores/actions/file.js create mode 100644 app/assets/javascripts/repo/stores/actions/tree.js create mode 100644 app/assets/javascripts/repo/stores/getters.js create mode 100644 app/assets/javascripts/repo/stores/index.js create mode 100644 app/assets/javascripts/repo/stores/mutation_types.js create mode 100644 app/assets/javascripts/repo/stores/mutations.js create mode 100644 app/assets/javascripts/repo/stores/mutations/branch.js create mode 100644 app/assets/javascripts/repo/stores/mutations/file.js create mode 100644 app/assets/javascripts/repo/stores/mutations/tree.js create mode 100644 app/assets/javascripts/repo/stores/state.js create mode 100644 app/assets/javascripts/repo/stores/utils.js diff --git a/app/assets/javascripts/api.js b/app/assets/javascripts/api.js index 242b3e2b990..d963101028a 100644 --- a/app/assets/javascripts/api.js +++ b/app/assets/javascripts/api.js @@ -16,6 +16,7 @@ const Api = { usersPath: '/api/:version/users.json', commitPath: '/api/:version/projects/:id/repository/commits', branchSinglePath: '/api/:version/projects/:id/repository/branches/:branch', + createBranchPath: '/api/:version/projects/:id/repository/branches', group(groupId, callback) { const url = Api.buildUrl(Api.groupPath) diff --git a/app/assets/javascripts/repo/components/new_branch_form.vue b/app/assets/javascripts/repo/components/new_branch_form.vue index eac43e692b0..ba7090e4a9d 100644 --- a/app/assets/javascripts/repo/components/new_branch_form.vue +++ b/app/assets/javascripts/repo/components/new_branch_form.vue @@ -1,18 +1,12 @@ diff --git a/app/assets/javascripts/repo/components/repo.vue b/app/assets/javascripts/repo/components/repo.vue index 788976a9804..0bc5271f95c 100644 --- a/app/assets/javascripts/repo/components/repo.vue +++ b/app/assets/javascripts/repo/components/repo.vue @@ -1,75 +1,42 @@ @@ -88,15 +55,6 @@ export default { - - + diff --git a/app/assets/javascripts/repo/components/repo_commit_section.vue b/app/assets/javascripts/repo/components/repo_commit_section.vue index 0d6259a37a8..37fc35f35c9 100644 --- a/app/assets/javascripts/repo/components/repo_commit_section.vue +++ b/app/assets/javascripts/repo/components/repo_commit_section.vue @@ -1,141 +1,97 @@