Skip to content

Adding support for multiple languages#11

Merged
proffapt merged 12 commits intoproffapt:mainfrom
YoganshSharma:main
Dec 10, 2022
Merged

Adding support for multiple languages#11
proffapt merged 12 commits intoproffapt:mainfrom
YoganshSharma:main

Conversation

@YoganshSharma
Copy link
Copy Markdown
Contributor

#8 Adding support for perl, lua, ruby and haskell.

Copy link
Copy Markdown
Owner

@proffapt proffapt left a comment

Choose a reason for hiding this comment

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

Are there no more options available for these langs?
Like compiled binary? debugger? Have you researched?

Comment thread run Outdated
Copy link
Copy Markdown
Owner

@proffapt proffapt left a comment

Choose a reason for hiding this comment

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

remove the --no-persistent flag the program is non-persistent by default

@proffapt
Copy link
Copy Markdown
Owner

proffapt commented Dec 9, 2022

#11 (comment)

@YoganshSharma as said compilation is designed to be persistent; no sense in making it non-persistent - like you compile and then delete what's the logic..

Comment thread run Outdated
Copy link
Copy Markdown
Owner

@proffapt proffapt left a comment

Choose a reason for hiding this comment

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

I have suggested some changes - let's see your views - if you find them ok - implement them directly from here.

Comment thread run Outdated
Comment thread run Outdated
Comment thread run Outdated
Comment thread run
YoganshSharma and others added 3 commits December 10, 2022 12:20
Co-authored-by: Arpit Bhardwaj <proffapt@pm.me>
Co-authored-by: Arpit Bhardwaj <proffapt@pm.me>
Co-authored-by: Arpit Bhardwaj <proffapt@pm.me>
Copy link
Copy Markdown
Contributor Author

@YoganshSharma YoganshSharma left a comment

Choose a reason for hiding this comment

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

I have reviewed the changes. It is ready to merge.

@proffapt
Copy link
Copy Markdown
Owner

LGTM.

@proffapt proffapt merged commit 39a4224 into proffapt:main Dec 10, 2022
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