summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch '20248-add-coverage-regex-in-job-yaml' into 'master'Douwe Maan2017-02-027-15/+86
|\ | | | | | | | | | | | | Add ability to define a coverage regex in the .gitlab-ci.yml Closes #20428 See merge request !7447
| * Make some other refinements to validation logicLeandro Camargo2017-01-251-2/+2
| |
| * Make more code improvements around the '/' stripping logicLeandro Camargo2017-01-252-8/+8
| |
| * Force coverage value to always be surrounded by '/'Leandro Camargo2017-01-253-17/+40
| |
| * Comply to more requirements and requests made in the code reviewLeandro Camargo2017-01-252-4/+3
| |
| * Simplify coverage setting and comply to some requests in code reviewLeandro Camargo2017-01-252-16/+6
| |
| * Fix wrong description for Coverage entry (in ruby comments)Leandro Camargo2017-01-251-1/+1
| |
| * Add specs to cover the implemented feature and fix a small bugLeandro Camargo2017-01-251-1/+1
| |
| * Add `coverage` to the Global config entry as wellLeandro Camargo2017-01-251-1/+4
| |
| * Change expected `coverage` structure for CI configuration YAML fileLeandro Camargo2017-01-254-9/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | Instead of using: `coverage: /\(\d+.\d+%\) covered/` This structure must be used now: ``` coverage: output_filter: /\(\d+.\d+%\) covered/` ``` The surrounding '/' is optional.
| * Comply to requests made in the review and adjust to the Entry/Node changesLeandro Camargo2017-01-254-8/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | This commit: * Turns `coverage_regex` into `coverage` entry in yml file * Fixes smaller requests from code reviewers for the previous commit * This commit is temporary (will be squashed afterwards) This commit does not (further commits will do though): * Add global `coverage` entry handling in yml file as suggested by Grzegorz * Add specs * Create changelog * Create docs
| * Add ability to define a coverage regex in the .gitlab-ci.ymlLeandro Camargo2017-01-255-5/+50
| | | | | | | | * Instead of using the proposed `coverage` key, this expects `coverage_regex`
* | Merge branch 'snippet-spam' into 'master' Rémy Coutable2017-02-022-2/+2
|\ \ | | | | | | | | | | | | | | | | | | Snippet spam Closes #26276 See merge request !8911
| * | Check public snippets for spamSean McGivern2017-02-022-2/+2
| | | | | | | | | | | | | | | | | | Apply the same spam checks to public snippets (either personal snippets that are public, or public snippets on public projects) as to issues on public projects.
* | | Merge branch '24462-reduce_ldap_queries_for_lfs' into 'master' Robert Speicher2017-02-011-3/+8
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Reduce hits to LDAP on Git HTTP auth by reordering auth mechanisms Closes #24462 See merge request !8752
| * | | Reduce hits to LDAP on Git HTTP auth by reordering auth mechanismsDrew Blessing2017-01-301-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We accept half a dozen different authentication mechanisms for Git over HTTP. Fairly high in the list we were checking user password, which would also query LDAP. In the case of LFS, OAuth tokens or personal access tokens, we were unnecessarily hitting LDAP when the authentication will not succeed. This was causing some LDAP/AD systems to lock the account. Now, user password authentication is the last mechanism tried since it's the most expensive.
* | | | Merge branch 'zj-format-chat-messages' into 'master' Kamil Trzciński2017-02-0116-182/+442
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | Format chat messages Closes #24917 and #25746 See merge request !8528
| * | | Improve styling of the new issue messagezj-format-chat-messagesZ.J. van de Weg2017-01-311-2/+2
| | | |
| * | | Incorporate feedbackZ.J. van de Weg2017-01-307-13/+38
| | | |
| * | | Rename presenters for consitencyZ.J. van de Weg2017-01-3011-36/+30
| | | |
| * | | Incorporate feedbackZ.J. van de Weg2017-01-309-180/+279
| | | |
| * | | Add help commandZ.J. van de Weg2017-01-303-8/+53
| | | |
| * | | Revert removing of some filesZ.J. van de Weg2017-01-303-0/+167
| | | |
| * | | Chat Commands have presentersZ.J. van de Weg2017-01-3015-336/+266
| |/ / | | | | | | | | | | | | This improves the styling and readability of the code. This is supported by both Mattermost and Slack.
* | | use destroy_allJames Lopez2017-01-311-3/+1
| | |
* | | remove old project members from projectJames Lopez2017-01-311-0/+4
| | |
* | | Merge branch ↵Kamil Trzciński2017-01-315-11/+4
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '25910-convert-manual-action-icons-to-svg-to-propperly-position-them' into 'master' Convert pipeline action icons to svg to have them propperly positioned Closes #25910 See merge request !8766
| * | | Removed unused methodFilipa Lacerda2017-01-262-7/+0
| | | |
| * | | Remove unneeded 'borderless' from icons nameFilipa Lacerda2017-01-254-5/+5
| | | |
| * | | added icons and fixed mini pipeline action dropdown iconsdimitrieh2017-01-254-4/+4
| | | |
* | | | Merge branch 'feature/gb/expose-commit-and-mr-pipelines-api' into 'master'Rémy Coutable2017-01-317-7/+7
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Expose pipelines API for commits and merge requests See merge request !8837
| * | | | Fix cycle analytics code after improving serializersGrzegorz Bizon2017-01-277-7/+7
| | | | |
* | | | | Use full group name/path in UI dropdownsDmitriy Zaporozhets2017-01-301-0/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | Merge branch 'fix/import-encrypt-atts' into 'master' Douwe Maan2017-01-303-7/+20
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ignore encrypted attributes in Import/Export Closes #24458 See merge request !8739
| * | | | | fix spec failurefix/import-encrypt-attsJames Lopez2017-01-301-1/+1
| | | | | |
| * | | | | programmatically remove encrypted attributes. Added relevant spec.James Lopez2017-01-301-0/+9
| | | | | |
| * | | | | rename method and added note to export file spec about new encrypted attributesJames Lopez2017-01-301-2/+2
| | | | | |
| * | | | | fix typoJames Lopez2017-01-301-1/+1
| | | | | |
| * | | | | Ignore encrypted attributes in Import/ExportJames Lopez2017-01-303-7/+11
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | * Regenerates tokens for all models that have them * Remove variables, since they are basically just storing encrypted data * Bumped version up to 0.1.6 * Updated related docs
* | | | | Improve performance of triggered chat commandszj-slow-service-fetchZ.J. van de Weg2017-01-301-3/+10
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the trigger endpoint is called, it has to find the right service for the given project. However, the old implementation did much more. For example, it build a list of the missing services on this project. This whole process took about 750ms _each time_. The current implementation is expected to perform 10x better, as it only searches in the current projects services. Given the service has to be configured anyway, this can be done.
* | | | update scripts and docs to reference the newly namespaced rake taskrelative-url-assetsMike Greiling2017-01-272-3/+3
| | | |
* | | | namespace assets rake tasks to gitlab:assets:*Mike Greiling2017-01-272-29/+47
| | | |
* | | | correct gzip files if they exist as wellMike Greiling2017-01-271-0/+19
| | | |
* | | | automatically correct CSS urls on assets:precompileMike Greiling2017-01-271-0/+10
| | | |
* | | | Move Gitlab::Shell and Gitlab::ShellAdapter files to lib/27280-fix-unititalized-constantRémy Coutable2017-01-272-0/+0
| | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | Move ApplicationSetting DEFAULTS to `.defaults` insteadRémy Coutable2017-01-271-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | This will avoid autoloading issues in the long term. Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | Merge branch '26943-fix-user-in-build-presenter' into 'master' Robert Speicher2017-01-272-0/+6
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | Don't override Ci::Build#user when presenting a build Closes #26943 See merge request !8668
| * | | Raise Gitlab::View::Presenter::CannotOverrideMethodError if presentee ↵26943-fix-user-in-build-presenterRémy Coutable2017-01-252-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | already respond to method Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | | Don't override presentee methods for Gitlab::View::Presenter::DelegatedRémy Coutable2017-01-241-1/+3
| | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | Make sure TraceReader uses Encoding.default_externalfix-trace-reader-encodingLin Jen-Shin2017-01-261-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Encoding.default_external was chosen over Encoding.default_internal because File.read is returning Encoding.default_external, therefore we should align with it. Alternatively, we could force both of them to be Encoding.default_internal. However, ideally this should be determined by different projects. For example, some projects might want to use an encoding different to what GitLab is using. This might not happen soon though. Closes #27052