Fixes #23895.

This was because the MR was a bit behind master and the error message
got updated in the meantime by 79aad815.
Signed-off-by: NRémy Coutable <remy@rymai.me>

Restore changes for api spec files

Fix error in rspec Users

Delete extra space Repositories-spec

Signed-off-by: NRémy Coutable <remy@rymai.me>

add docs and tests - add additional validation
allow move without content
updated response

https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6305#note_15230024

This change simplifies a Pipeline processing by introducing a special new status: created.
This status is used for all builds that are created for a pipeline.
We are then processing next stages and queueing some of the builds (created -> pending) or skipping them (created -> skipped).
This makes it possible to simplify and solve a few ordering problems with how previously builds were scheduled.
This also allows us to visualise a full pipeline (with created builds).

This also removes an after_touch used for updating a pipeline state parameters.
Right now in various places we explicitly call a reload_status! on pipeline to force it to be updated and saved.

So we have raw_diffs too

- Parameter validation as ISO8601 format

When fetching commit comments via API, the comments were not ordered,
but just returned in the order Postgresql finds them. Now the API always
returns comments in chronological order.

Signed-off-by: NJeroen van Baarsen <jeroenvanbaarsen@gmail.com>

Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>