Skip to content

Conversation

@borisonekenobi
Copy link

@borisonekenobi borisonekenobi commented Dec 3, 2025

Adding a language

  • The code displays "Hello World" (tio.run may help for testing)
  • I have no association with the language
  • There are no copyright issues with this code
  • The language has not been added prior to this pull request
  • I have updated the README

Link to programming language:

Copilot AI review requested due to automatic review settings December 3, 2025 15:48
Copilot finished reviewing on behalf of borisonekenobi December 3, 2025 15:52
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds six English programming language dialects to the repository: English, ~English, !English, ^English, English+, and enGLish. However, several issues need to be addressed before this PR can be merged.

  • Adds 5-6 new English language variants (though ~English is a duplicate)
  • Updates the total language count from 1007 to 1013
  • Adds README entries for all six languages

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
readme.md Updates language count and adds entries for six English language variants, but has ordering and duplicate issues
e/English Adds English language "Hello World" implementation
e/English+ Adds English+ language "Hello World" implementation
e/enGLish (2) Adds enGLish language with incorrect "(2)" suffix in filename
e/!English Adds !English language but placed incorrectly (should be in special characters section)
e/^English Adds ^English language but placed incorrectly (should be in special characters section)
e/~English Duplicate of existing #/~English file, should be removed

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +333 to +334
* [!English](e/%21English)
* [^English](e/%5EEnglish)
Copy link

Copilot AI Dec 3, 2025

Choose a reason for hiding this comment

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

Languages starting with special characters should be listed in the special characters section at the top of the README (lines 13-51) with paths to the # directory, not in the middle of the alphabetical listing with paths to the e directory.

For example, !English, ^English, and ~English should be placed in the top section alongside other languages like !, @tention!, and the existing ~English entry (line 33).

Copilot uses AI. Check for mistakes.
* [Emoticon](e/Emoticon.emoticon)
* [Emotinomicon](e/Emotinomicon)
* [English](e/English)
* [enGLish (2)](e/enGLish%20%282%29)
Copy link

Copilot AI Dec 3, 2025

Choose a reason for hiding this comment

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

The filename should be enGLish without the "(2)" suffix, as there is no other enGLish language in the repository. The "(2)" suffix is only used when there's already another language with the same name.

Suggested change
* [enGLish (2)](e/enGLish%20%282%29)
* [enGLish](e/enGLish)

Copilot uses AI. Check for mistakes.
Comment on lines +358 to +359
* [enGLish (2)](e/enGLish%20%282%29)
* [English+](e/English%2B)
Copy link

Copilot AI Dec 3, 2025

Choose a reason for hiding this comment

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

The alphabetical ordering is incorrect. English+ (line 359) should come before enGLish (2) (line 358). The correct order should be: English, English+, enGLish.

Suggested change
* [enGLish (2)](e/enGLish%20%282%29)
* [English+](e/English%2B)
* [English+](e/English%2B)
* [enGLish (2)](e/enGLish%20%282%29)

Copilot uses AI. Check for mistakes.
@borisonekenobi
Copy link
Author

borisonekenobi commented Dec 3, 2025

Notes for Reviewer:

  • Some of the dialects start with, not the letter e, but I wasn't sure if they should or shouldn't be put in the #/ directory, so I've put everything in the e/ directory
  • I couldn't find anything about file extensions for any of the languages, so I've left all of the files extentionless
  • English and enGLish have the same file name according to Windows, so I've added the (2) suffix to enGLish
  • All of these are in one commit, can be changed to be separate commits for each file

Let me know if any of these need to be changed before being merged, will be happy to make the changes!

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.

1 participant