We had to change the command for openning pipeline in the browser
to support detached pipelines.

All the data we need are in the API response that lists pipelines.

Prefer detached pipelines and only report normal branch pipeline when
there isn't any detached pipeline (MR pipeline).

We also change the logic to fetch MR and pipeline at the same time
because we need to know MR anyway to be able to get detached
pipeline.

Refactoring status_bar to use both pipeline and MR from the response
is too difficult to squeeze in this change. We'll have to do it later.

This reduces the need for readers to keep a variable
in their head to read 50 LOC.

By relying on an exception, we are able to remove several checks
for failed project fetching. This also saves one extra call to
the /project endpoint. Additionally, this commit replaces a long
if statement with an early return.

The `pipelineGitRemoteName` setting is used to say:
"Use this remote for getting pipelines"
This purpose of this configuration is to mark a fork. Then
you can read issues and MRs from the main project and
pipelines from your fork.

The service can be made simpler by moving the pipeline specific remote
out of the service and exposing a more general method.

Extracting configuration related logic came up in reviews multiple times.
Also, the VS Code logic returns missing properties as null.
This behaviour is not ideal because null function arguments overrdie the
default function arguments.

Related discussions:
- https://gitlab.com/gitlab-org/gitlab-vscode-extension/-/merge_requests/115#note_432956679

This fixes a minor bug where if user hasn't configured their
instaenceUrl (presumably they use the default gitlab.com),
the webview is going to put `null` in the links. This fix
makes sure we use the same logic as the rest of the app.

test: increase integration test timeout

See merge request gitlab-org/gitlab-vscode-extension!184

refactor(status bar): extract status_bar module into a class

See merge request gitlab-org/gitlab-vscode-extension!181

test(integration): reduce test flakiness by replacing globalState

See merge request gitlab-org/gitlab-vscode-extension!180

test(for_current_branch): fix hardcoded date breaking tests

See merge request gitlab-org/gitlab-vscode-extension!179

fix: support displaying users without avatars

See merge request gitlab-org/gitlab-vscode-extension!178

Update GitLab CI variables (used for autocompletion) + add docs

See merge request gitlab-org/gitlab-vscode-extension!177

fix(sidebar): log error when fetching items

See merge request gitlab-org/gitlab-vscode-extension!175

This rule is critical. We must have a guarantee that when there is
a try/catch around a return statement, we will catch the error.

test(user agent): fix broken user-agent test

See merge request gitlab-org/gitlab-vscode-extension!176

fix(side tree): for current branch not working for multiroot projects

See merge request gitlab-org/gitlab-vscode-extension!174

feat: try to get MR diff conent from local git before fetching from API

See merge request gitlab-org/gitlab-vscode-extension!173

This is an optimisation where we check the local git repository
and only when the file in given version doesn't exist there
we'll call the GitLab API to retrieve the file content.

docs: merge request reviews feature documentation

See merge request gitlab-org/gitlab-vscode-extension!170