- 30 Sep, 2016 1 commit
-
-
Jacob Vosmaer authored
This side-steps the need for mutexes and whatnot.
-
- 29 Sep, 2016 1 commit
-
-
Patricio Cano authored
Fix race condition that can be triggered if the token expires right after we retrieve it, but before we can set the new expiry time.
-
- 28 Sep, 2016 3 commits
-
-
Patricio Cano authored
Reset expiry time of token, if token is retrieved again before it expires.
-
Rémy Coutable authored
Changes include: - Ensure Member.add_user is not called directly when not necessary - New GroupMember.add_users_to_group to have the same abstraction level as for Project - Refactor Member.add_user to take a source instead of an array of members - Fix Rubocop offenses - Always use Project#add_user instead of project.team.add_user - Factorize users addition as members in Member.add_users_to_source - Make access_level a keyword argument in GroupMember.add_users_to_group and ProjectMember.add_users_to_projects - Destroy any requester before adding them as a member - Improve the way we handle access requesters in Member.add_user Instead of removing the requester and creating a new member, we now simply accepts their access request. This way, they will receive a "access request granted" email. - Fix error that was previously silently ignored - Stop raising when access level is invalid in Member, let Rails validation do their work Signed-off-by: Rémy Coutable <remy@rymai.me>
-
Markus Koller authored
Signed-off-by: Roger Meier <r.meier@siemens.com>
-
- 27 Sep, 2016 6 commits
-
-
Ahmad Sherif authored
-
Ahmad Sherif authored
-
Ahmad Sherif authored
* No need to re-fetch issues from GH to read their labels, the labels are already there from the index request. * No need to look up labels on the database for every application, so we cache them.
-
Ahmad Sherif authored
This should avoid having large memory growth when importing GitHub repos with lots of resources.
-
Patricio Cano authored
Do not regenerate the `lfs_token` every time `git-lfs-authenticate` is called, instead return the saved token if one is present.
-
James Lopez authored
-
- 26 Sep, 2016 1 commit
-
-
James Lopez authored
Fixes issue with rails reserved keyword type exporting/importing services. Also fixed CustomIssueTrackerService title setter and added relevant specs.
-
- 23 Sep, 2016 1 commit
-
-
Grzegorz Bizon authored
-
- 22 Sep, 2016 3 commits
-
-
Nick Thomas authored
This commit changes the revisions used for diffs. The current behaviour is to show all changes between current tip of master and tip of the MR, rather than matching the output of the web frontend (which just shows the changes in the MR). Switching from start_sha to base_sha fixes this.
-
Grzegorz Bizon authored
-
Jacob Vosmaer authored
-
- 21 Sep, 2016 2 commits
-
-
James Lopez authored
-
Timothy Andrew authored
- Pluralize summary titles - Remove the `run_query` method - always return sql strings from the `date_time_sql` methods
-
- 20 Sep, 2016 10 commits
-
-
Timothy Andrew authored
-
Timothy Andrew authored
- Don't use `TableReferences` - using `.arel_table` is shorter! - Move some database-related code to `Gitlab::Database` - Remove the `MergeRequest#issues_closed` and `Issue#closed_by_merge_requests` associations. They were either shadowing or were too similar to existing methods. They are not being used anywhere, so it's better to remove them to reduce confusion. - Use Rails 3-style validations - Index for `MergeRequest::Metrics#first_deployed_to_production_at` - Only include `CycleAnalyticsHelpers::TestGeneration` for specs that need it. - Other minor refactorings.
-
Dan Dunckel authored
-
Timothy Andrew authored
And `scss_lint`
-
Timothy Andrew authored
- Move things common to `Issue` and `MergeRequest` into `Issuable` - Move more database-specific functions into `Gitlab::Database` - Indentation changes and other minor refactorings.
-
Timothy Andrew authored
1. Change multiple updates to a single `update_all` 2. Use cascading deletes 3. Extract an average function for the database median. 4. Move database median to `lib/gitlab/database` 5. Use `delete_all` instead of `destroy_all` 6. Minor refactoring
-
Kamil Trzcinski authored
-
James Lopez authored
- Fixed and added specs. - Added different namespace options depending on user privilages - Updated docs.
-
Kamil Trzcinski authored
-
Robert Speicher authored
-
- 19 Sep, 2016 12 commits
-
-
Dan Dunckel authored
-
Kamil Trzcinski authored
Revert "Revert all changes introduced by https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6043" This reverts commit 6d43c95b.
-
Kamil Trzcinski authored
-
Kamil Trzcinski authored
-
Kamil Trzcinski authored
-
Nick Thomas authored
-
Kamil Trzcinski authored
-
Kamil Trzcinski authored
-
Kamil Trzcinski authored
Squashed commits: [f9a9315] Use : to test invalid environment name
-
Grzegorz Bizon authored
-
Kamil Trzcinski authored
-