Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add Phel language#6282

Open
JesusValeraDev wants to merge 5 commits intogithub-linguist:mainfrom
JesusValeraDev:add-phel-language
Open

Add Phel language#6282
JesusValeraDev wants to merge 5 commits intogithub-linguist:mainfrom
JesusValeraDev:add-phel-language

Conversation

@JesusValeraDev
Copy link
Copy Markdown

@JesusValeraDev JesusValeraDev commented Feb 10, 2023

📚 Description

There are more than 480 files in Github using the .phel extension.

Some remarkable projects:

Plugins for different applications

📓 Checklist:

The last two points are not selected because they use a Common Lisp grammar & the extension is unique (.phel).


Please let me know if something is missing. I'd be glad to keep improving the PR if necessary! 🤞🏼

@JesusValeraDev JesusValeraDev requested a review from a team as a code owner February 10, 2023 16:20
Copy link
Copy Markdown

@Chemaclass Chemaclass left a comment

Choose a reason for hiding this comment

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

Great idea! Highlighting phel files using common lisp would be a great improvement, especially people will be able to understand the code easier when reading it directly from GitHub 🧠💡

Copy link
Copy Markdown
Member

@lildude lildude left a comment

Choose a reason for hiding this comment

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

Popularity is not sufficient for inclusion yet, but that doesn't mean you can't address the identified issues in the mean time.

See inline comment.

Please remove the core.phel file. It's massive and waaaay too big.

Please also remove the sample.phel file as this is clearly a "Hello world" sample and not a real-world example.

Comment thread lib/linguist/languages.yml Outdated
@lildude lildude dismissed their stale review July 10, 2024 08:59

Changed applied. Pending popularity now.

@JesusValeraDev JesusValeraDev requested a review from a team as a code owner September 23, 2025 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants