Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Universal Code Search blog post for GitLab #662

Merged
merged 3 commits into from
Mar 13, 2020

Conversation

ryan-blunden
Copy link
Contributor

@ryan-blunden ryan-blunden commented Mar 13, 2020

Blog post with embedded video for configuring GitLab as a code host.

@ryan-blunden ryan-blunden force-pushed the rb-universal-code-search-gitlab branch from 914808f to ce2b440 Compare March 13, 2020 00:26
blogposts/universal-code-search-gitlab.md Outdated Show resolved Hide resolved

This screencast (with accompanying audio) shows how to [connect and configure](https://docs.sourcegraph.com/admin/external_service/gitlab) Sourcegraph Universal Code Search for public and private repositories on GitLab.com and GitLab CC/EE instances

In addition to connection configuration, it discusses important factors for determining which repositories Sourcegraph will index, such as excluding archived repositories, as well as repositories based on naming patterns such `deploy-`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
In addition to connection configuration, it discusses important factors for determining which repositories Sourcegraph will index, such as excluding archived repositories, as well as repositories based on naming patterns such `deploy-`.
In addition to configuring the Sourcegraph-GitLab connection, the screencast discusses important factors for setting up your Sourcegraph instance, such as determining which repositories Sourcegraph will index, excluding archived repositories, or including repositories based on naming patterns (e.g.,`deploy-`).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I incorporated most of these changes, thanks!


See the [Sourcegraph GitLab documentation](https://docs.sourcegraph.com/admin/external_service/gitlab) for full configuration options, including [enabling repository permissions](https://docs.sourcegraph.com/admin/external_service/gitlab#repository-permissions).

The script used in the screencast to search for repositories to ignore is available as a [GitLab snippet](https://gitlab.com/snippets/1952534).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This reads awkwardly but I don't have an improvement. There are some typos in the comment of your script, and I think there's a stray 'f' on line 14.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, fixed the typos in the script and I'll re-work this sentence.


See the [Sourcegraph GitLab documentation](https://docs.sourcegraph.com/admin/external_service/gitlab) for full configuration options, including [enabling repository permissions](https://docs.sourcegraph.com/admin/external_service/gitlab#repository-permissions).

The script used in the screencast to search for repositories to ignore is available as a [GitLab snippet](https://gitlab.com/snippets/1952534).
Copy link
Contributor

@vanesa vanesa Mar 13, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The script used in the screencast to search for repositories to ignore is available as a [GitLab snippet](https://gitlab.com/snippets/1952534).
You can use this [Python script](https://gitlab.com/snippets/1952534) used in the screencast to search for repositories you want to exclude.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I took your idea and incorporated it into a new version of this sentence. Thanks!

Copy link
Contributor

@vanesa vanesa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than the rewording and the comment typo in the GitLab snippet (Script that be be used ), approved! Great work. Can't wait to share this on Twitter.

Ryan Blunden and others added 2 commits March 13, 2020 11:16
Co-Authored-By: Christina Forney <christina@sourcegraph.com>
@ryan-blunden ryan-blunden merged commit 68ea07c into master Mar 13, 2020
@ryan-blunden ryan-blunden deleted the rb-universal-code-search-gitlab branch March 13, 2020 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants