Skip to content

Commit

Permalink
Remove ldml extension from Lasso (#4615)
Browse files Browse the repository at this point in the history 
* Remove ldml extension from Lasso

Usage is far too low for inclusion and clashes with the also too low Unicode ldml files (which are XML). The clash however means the Unicode ldml files are identified as Lasso instead of data by the XML classifier.

Fixes #4597

* Remove Lasso ldml sample
  • Loading branch information
@lildude
lildude committed Aug 21, 2019
1 parent 4b3448c commit d2e0956
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 8,343 deletions.
1 change: 0 additions & 1 deletion lib/linguist/languages.yml
View file
Expand Up @@ -2685,7 +2685,6 @@ Lasso:
- ".las"
- ".lasso8"
- ".lasso9"
- ".ldml"
tm_scope: file.lasso
aliases:
- lassoscript
Expand Down

1 comment on commit d2e0956

@Alhadis
Copy link
Collaborator

@Alhadis Alhadis commented on d2e0956 Aug 22, 2019 •
edited

Choose a reason for hiding this comment

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

You forgot to remove the .ldml sample as well (running script/list-grammars alerted me to it).

Don't worry, I'll be sure to remove it in the PR I'm prepping with the other fixes.

EDIT: Wait, scratch that… you did remove it. 😕 My setup might be a bit screwed up... 😰

Please sign in to comment.