Use Gitlab::PushOptions for `ci.skip` push option

Previously the raw push option Array was sent to Pipeline::Chain::Skip.

This commit updates this class (and the chain of classes that pass the
push option parameters from the API internal `post_receive` endpoint to
that class) to treat push options as a Hash of options parsed by
GitLab::PushOptions.

The GitLab::PushOptions class takes options like this:

    -o ci.skip -o merge_request.create -o merge_request.target=branch

and turns them into a Hash like this:

    {
      ci: {
        skip: true
      },
      merge_request: {
        create: true,
        target: 'branch'
      }
    }

This now how Pipeline::Chain::Skip is determining if the `ci.skip` push
option was used.

1 files changed, 1 insertions, 1 deletions

diff --git a/lib/api/internal.rb b/lib/api/internal.rb
index 75202fa953c..ee1fb465608 100644
--- a/lib/api/internal.rb
+++ b/lib/api/internal.rb
@@ -262,7 +262,7 @@ module API
         push_options = Gitlab::PushOptions.new(params[:push_options])
 
         PostReceive.perform_async(params[:gl_repository], params[:identifier],
-          params[:changes], params[:push_options].to_a)
+          params[:changes], push_options.as_json)
 
         if (mr_options = push_options.get(:merge_request))
           begin