Skip to content

GitLab

Explore

Sign in
Register

Primary navigation

Project

S

sourcegraph
- Activity
- Members
- Labels
- Issues
- Issue boards
- Milestones
- Wiki
- Snippets
- Package registry
- Model registry
- Environments
- Terraform modules
- Incidents
- Value stream analytics
- Model experiments

Snippets Groups Projects

!40783

syntax-highlighter: Use sourcegraph/syntect, not slimsag/syntect.

Review changes
Download
Patches
Plain diff

Merged syntax-highlighter: Use sourcegraph/syntect, not slimsag/syntect.

vg/use-sg-syntect into main

Overview 2
Commits 1
Pipelines 0
Changes 4

Merged syntax-highlighter: Use sourcegraph/syntect, not slimsag/syntect.

Warren Giffordrequested to merge

vg/use-sg-syntect into main Aug 24, 2022

Overview 2
Commits 1
Pipelines 0
Changes 4

Created by: varungandhi-src

@tjdevries can you double-check that there's no behavior change with this? I tested a few languages visually.

Test plan

The tests in sourcegraph/syntect are passing. Visually inspected the highlighting for different languages like JavaScript, Ruby and Markdown with this change -- didn't see any regression.

Merge request reports

Activity

Filter activity

Approvals
Assignees & reviewers
Comments (from bots)
Comments (from users)
Commits & branches
Edits
Labels
Lock status
Mentions
Merge request status
Tracking

Please register or sign in to reply

0 Assignees

None

Select assignees

0 Reviewers

None

Request review from

Loading

Labels

1

1

Select labels

Manage project labels

Milestone

None

None

None

Time tracking

No estimate or time spent

1

1 Participant

There are no commits yet

Push commits to the source branch or add previously merged commits to review them.

Loading