delta/gitlab/gitlab-ce.git/spec/models/commit_spec.rb, branch frozen_string_lib_2

Fix inline rendering of SVGs from current repo

2019-08-01T22:34:20+00:00

Changes generated URL to raw instead of blob

Speed up commit loads by disabling BatchLoader replace_methods

2019-06-13T21:21:23+00:00

We've seen a significant performance penalty when using
`BatchLoader#__replace_with!`. This defines methods on the batch loader
that proxy to the 'real' object using send. The alternative is
`method_missing`, which is slower.  However, we've noticed that
`method_missing` can be faster if:

1. The objects being loaded have a large interface.
2. We don't call too many methods on the loaded object.

In production, we've seen the rendering times of the merge request
widget increase as a result of loading commit data. BatchLoader attempts
to call replace_methods on the lazy object, but this has a significant
performance penalty. Disabling this mode
(https://github.com/exAspArk/batch-loader/pull/45) appears to cut load
times by about 50% for MergeRequestsController#show.

Relates to https://gitlab.com/gitlab-com/gl-infra/infrastructure/issues/6941

Add # frozen_string_literal to spec/models

2019-04-01T01:37:54+00:00

Adds `# frozen_string_literal: true` to spec/models ruby files

Add back Rugged support for retrieving a commit tree entry

2019-03-09T00:01:59+00:00

This brings back some of the changes in
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/20176/diffs.

We discovered another N+1 that hits Gitaly `TreeEntry` via the
`RelativeLinkFilter`:
https://gitlab.com/gitlab-org/gitlab-ce/issues/58657. When a blob is
loaded with many relative links, `TreeEntry` is called for each link to
scan the URI type.

There are multiple paths that hit Gitaly `TreeEntry`, and
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/25706 did not
cover all cases. This commit covers another common use case.

For users using Gitaly on top of NFS, accessing the Git data directly
via Rugged may be faster than going through than Gitaly. This merge
request introduces the feature flag `rugged_commit_tree_entry` to
activate the Rugged method.

[master] Pipelines section is available to unauthorized users

2019-01-31T15:52:50+00:00

Truncate merge request titles with periods instead of ellipsis

2018-12-07T05:51:51+00:00

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/54862

Match users better by their private commit email

2018-11-19T12:45:07+00:00

Private commit emails were introduced in !22560, but some parts of
GitLab were not updated to take account of them. This commit adds
support in places that were missed.

Merge branch 'fix-committer-typo' into 'master'

2018-09-25T17:08:23+00:00

Fix committer typo

Closes #51808

See merge request gitlab-org/gitlab-ce!21899

Fix committer typo

2018-09-25T08:51:07+00:00

Use `Gitlab::SafeRequestStore` in more places

2018-09-24T19:11:27+00:00

Even if it doesn’t save lines of code, since people will tend to use
code they’ve seen. And `SafeRequestStore` is safer since you
don’t have to remember to check `RequestStore.active?`.