-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add support for the Rust language #102
base: development
Are you sure you want to change the base?
Conversation
Currently only running testcases with compilation flags is allowed in haskell. Stress Testing has not been added due to difficulty with System.Random package
Add Haskell support and fix testcase parsing
Implemented support for the Rust programming language, which previously had not been available in the extension. The necessary changes have been applied.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change your branch to "development" and make a pull request for IEEE-NITK:development. I will merge dev branch to master branch if the code passes all reviews
package.json
Outdated
@@ -542,6 +549,7 @@ | |||
"cheerio": "^1.0.0-rc.10", | |||
"child_process": "^1.0.2", | |||
"fs": "^0.0.1-security", | |||
"node-fetch": "^2.6.1" | |||
"node-fetch": "^2.6.1", | |||
"problem-matcher": "^1.1.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove "problem matcher" dependency
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The dependency has been removed. (397baf0)
@@ -90,6 +90,17 @@ export const runTestsWithTimeout = async ( | |||
? "java.exe" | |||
: "java"; | |||
break; | |||
case CompilationLanguages.rust: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This entire block of code is not necessary. You can move "case CompilationLanguages.rust:" to below the c/cpp/haskell case block
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have made the necessary adjustments. (a2a2175)
I added the rust compilation code to an existing case instead of creating a new one, as it could have been included in the c/cpp/haskell case, to avoid unnecessary duplication.
As requested, I switched the branch from master to development. Can you please take a look? |
Implemented support for the Rust programming language, which previously had not been available in the extension. The necessary changes have been applied.