Marcel Amirault authored 3 years ago and James Lopez committed 3 years ago

We don't need to hardcode to master, we should use the
default branch CI/CD variable

This changes the GraphQL markdown documentation to have links to
types.

This reverts merge request !46018

This reverts merge request !55982

Marcel Amirault authored 3 years ago and Shinya Maeda committed 3 years ago

We don't need to hardcode to master, we should use the
default branch CI/CD variable

Andy Soiron authored 3 years ago and Peter Leitzen committed 3 years ago

The worker would otherwise send to the same email
address multiple times

In https://gitlab.com/gitlab-com/gl-infra/production/-/issues/3894, we
found that query plans using `base_and_descendants` as a subquery in an
`IN ()` clause may trigger a PostgreSQL v11 query planner bug, resulting
in a significant performance degradation.

https://gitlab.com/gitlab-org/gitlab/-/merge_requests/56078 works around
this issue by moving the clause into a INNER JOIN, but finders may have
to take advantage of this by calling `Group#all_projects` instead of
`base_and_descendants`.

This is a similar fix to
https://gitlab.com/gitlab-org/gitlab/-/merge_requests/56346.

Relates to https://gitlab.com/gitlab-org/gitlab/-/issues/324220

Jan Provaznik authored 3 years ago and Charlie Ablett committed 3 years ago

Label arguments and related logic (arguments validation) is needed
for epic boards too, so these arguments are moved to
ScopedBoardMutation and issue-specific arguments are renamed left in
ScopedIssueBoardArguments.

Justin Boyson authored 3 years ago and David O'Regan committed 3 years ago

Updates specs to check for correct string parsing

Client ID will have one of these values:
- user/user.id
- runner/runner.id
- ip/request.ip

Steve Mokris authored 3 years ago and Mark Chao committed 3 years ago

- Allow access to ping endpoint without a token
- Allow access to download endpoints without a token, if the project
  is public
- Instead of throwing UnauthorizedError when no token is provided,
  continue with anonymous privileges
- Update tests to reflect the above changes
- Add test for downloading without a token

Adds an API endpoint which allows for deletion of old, stopped,
review app environments.

Including maintainer review commits:

Fix variables names in HAML templates
Regenerate the locales file
Add context to locales phrases
Bring back the form area
Fix disable_merge_trains_setting spec
Remove incorrect prefix from some i18n strings

In this commit we fix a typo (`reponse` vs. `response`) and
also the method call `response_api_error!` that was lacking
one argument.

- Also renamed graphql query since we're fetching
  more than just userPermissions now.

This change remove the block class that creates adds padding and a
border in favor of utility classes.

Jaime Martinez authored 3 years ago and Achilleas Pipinellis committed 3 years ago

Adds multuple flags used to configure Pages API-based configuration
cache's behavior to the documentation. Examples of modifying these
settings are also included.

Related to https://gitlab.com/gitlab-org/gitlab-pages/-/issues/520.

Jörg Behrmann authored 3 years ago and Evan Read committed 3 years ago

This change brings it in line with the order during install and makes
the workhorse installation not error out, since it depends on new gems
being installed first

- it also fixes broken assignees_label

Andy Soiron authored 3 years ago and Alexis Kalderimis committed 3 years ago

This uses HLLRedisCounter to log usage
for individual users of slack service
each time a individual user triggers an action
it should increase the counter for this action

David Kim authored 3 years ago and Mayra Cabrera committed 3 years ago

We currently have Assignee information only on text version so we'll
follow the same reasoning to be consistent.

Use the grouped_issues_list component to add
section headers to the test report MR widget.

Also refactored the test issue body to handle
it's own status icon and use gitlab-ui more.

Refactored the test issue body spec to use up
to date standards.

Jarek Ostrowski authored 4 years ago and Enrique Alcántara committed 3 years ago

MR: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/55522

Giorgenes Gelatti authored 4 years ago and Steve Abrams committed 3 years ago

Updates composer cache upon
package creation and deletion.

Nicolò Mezzopera authored 3 years ago and Olena Horal-Koretska committed 3 years ago

- source
- tests

hstore can only store string keys. So using symbols as a default causes
a confusing situation where it is a symbol until the model is reloaded
where it then loads as a string.

This reverts commit 644e74d8.

Michał Zając authored 3 years ago and Adam Hegyi committed 3 years ago

In https://gitlab.com/gitlab-org/gitlab/-/merge_requests/46483 we have
started calculating UUIDv5 for new Findings instead of using just
time-based UUIDs. This will schedule a background data migration
to Recalculate all UUIDs so our Feedback <-> Finding matching is
more stable.

When processing commits to include in a changelog, we may encounter
commits without a description (= they only have a subject). This results
in Commit#description being nil, resulting in a NoMethodError when we
try to extract a revert SHA from that description.

This commit fixes this by just not matching if the description is nil.
This is fine since commits without a description are never included, due
to a lack of a trailer tag.

This fixes https://gitlab.com/gitlab-org/gitlab/-/issues/323998

The standard is to use CI/CD not CI / CD

Raimund Hook authored 3 years ago and Mikołaj Wawrzyniak committed 3 years ago

Signed-off-by: Raimund Hook <raimund.hook@exfo.com>

This pulls in https://github.com/fog/fog-google/pull/517 to deal with
the log spam in https://gitlab.com/gitlab-org/gitlab/-/issues/320790.