Skip to content

Add Java to language directory#15

Open
kjw142857 wants to merge 3 commits into
mainfrom
add-java
Open

Add Java to language directory#15
kjw142857 wants to merge 3 commits into
mainfrom
add-java

Conversation

@kjw142857

Copy link
Copy Markdown

Adds Java as one of the languages in the language directory.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request adds Java language support by defining a new language object and registering it in the central languages list. The feedback suggests reordering the language array alphabetically and adding trailing commas to improve code maintainability and diff readability.

Comment thread src/languages.ts
Comment on lines +9 to +10
pythonLanguage,
javaLanguage

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

To maintain consistency with the import order and improve maintainability, the languages in the languages array should be kept in alphabetical order. Additionally, adding a trailing comma is a best practice in TypeScript to ensure cleaner diffs when adding new elements in the future.

Suggested change
pythonLanguage,
javaLanguage
javaLanguage,
pythonLanguage,

@martin-henz martin-henz self-requested a review April 1, 2026 01:38

@martin-henz martin-henz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The language evaluator url is missing.

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.

2 participants