Skip to content

added EKL syntax #9233

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.

Sign up for GitHub

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

Open
kaessajidi wants to merge 2 commits into
base: master
Choose a base branch
from
Open

added EKL syntax #9233

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
db96118
added EKL syntax
kaessajidi Oct 25, 2025
9630a9b
removed package.json
kaessajidi Oct 29, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions repository/e.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,18 @@
{
"schema_version": "3.0.0",
"packages": [
{
"details": "https://github.com/kaessajidi/ekl-sublim-text",
"name": "Dassault Systemes EKL Syntax",
Copy link
Collaborator

@braver braver Oct 29, 2025

Choose a reason for hiding this comment

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

Suggested change
"name": "Dassault Systemes EKL Syntax",
"name": "Dassault Systemes EKL",

Language support packages don't need to have "syntax" or something like it in the name, since typically it does more than just the syntax: in your case it also provides completions (which you might want to specify in the labels).

If you want to call it Dassault Systemes EKL and not just EKL, it should go into the d.json file instead, an in alphabetical order there.

Copy link
Collaborator

@braver braver Nov 16, 2025

Choose a reason for hiding this comment

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

If you want to call it Dassault Systemes EKL and not just EKL, it should go into the d.json file instead, an in alphabetical order there.

You missed this point I'm afraid.

Copy link
Collaborator

@braver braver Dec 8, 2025

Choose a reason for hiding this comment

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

👋🏻 your PR is not mergeable in its current state.

"author": "Kamal Essa",
"labels": ["language syntax"],
"releases": [
{
"sublime_text": "*",
"tags": true
}
]
},
{
"details": "https://github.com/glyph/E-Max",
"releases": [
Expand Down
Loading