Skip to content
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.

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

Feature: Added TypeScript Support #3 #152

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

Conversation

dhruvrjain
Copy link

Solves issue #3

Title: Add TypeScript language support

Description:

This pull request adds support for the TypeScript programming language to our project. The implementation includes test cases for basic functionality and standard input/output operations.

Changes made:

  • Added TypeScript language execution support in the main execution engine
  • Implemented "Hello World" test case for TypeScript
  • Updated relevant configuration files to include TypeScript
  • Added TypeScript compilation step to the execution process

New test cases:

  • "typescript : hello world" - Tests basic TypeScript program execution

These test cases follow the existing structure and format of other language test cases in the project.

Testing:

  • All new test cases pass successfully
  • Existing test cases for other languages remain unaffected

Dependencies:

  • This feature requires Node.js and the TypeScript compiler (tsc) to be installed on the system running the project

@dhruvrjain
Copy link
Author

@anipnwr7777 Please review and let me know if any changes or additional information is needed.

Comment on lines +214 to +225
{
name: 'typescript : hello world',
reqObject: {
language: 'typescript',
script: 'console.log("hello world");',
},
expectedResponse: {
val: 'hello world\n',
status: 200,
error: 0,
},
},
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hey can you please add a test case here that takes an input and then logs the same on console ?

Copy link
Author

Choose a reason for hiding this comment

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

Sure. I'll start working on it and let you know when it is done.

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