Skip to content

Support JetBrains colour scheme (.icls) files#7851

Merged
lildude merged 3 commits intogithub-linguist:mainfrom
hearsilent:dev/icls
Mar 16, 2026
Merged

Support JetBrains colour scheme (.icls) files#7851
lildude merged 3 commits intogithub-linguist:mainfrom
hearsilent:dev/icls

Conversation

@hearsilent
Copy link
Contributor

@hearsilent hearsilent commented Mar 14, 2026

Description

This PR adds JetBrains colour scheme (.icls) files to the list of supported XML extensions.

Checklist

  • I am adding a new extension to a language.
    • The new extension is used in hundreds of repositories on GitHub.com
      ~11.1k search results
    • I have included a real-world usage sample.
      • Deep_Ocean.icls: Source | MIT-licensed
    • I have included a change to the heuristics.

@hearsilent hearsilent requested a review from a team as a code owner March 14, 2026 12:05
@Alhadis Alhadis changed the title Support JetBrains color scheme (.icls) Support JetBrains colour scheme (.icls) files Mar 15, 2026
Alhadis
Alhadis previously approved these changes Mar 15, 2026
Copy link
Collaborator

@Alhadis Alhadis left a comment

Choose a reason for hiding this comment

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

Usage checks out; at least ~11.1k .icls files lack an XML declaration, meaning that Linguist's XML strategy isn't catching these files. Hence, adding this to XML's file-extensions list is warranted.

Not sure why the tests are failing, though. 🤔 EDIT: Never mind, I figured it out. I've pushed a fix to the PR.

@lildude
Copy link
Member

lildude commented Mar 16, 2026

LGTM. Thanks.

Important

The changes in this PR will not appear on GitHub until the next release has been made and deployed. See here for more details.

@lildude lildude added this pull request to the merge queue Mar 16, 2026
Merged via the queue into github-linguist:main with commit 1549797 Mar 16, 2026
5 checks passed
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.

3 participants