Skip to content

Add merge.js equivalent written in python #3

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

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

Conversation

Zvitorep
Copy link

As said in the title

@lennyerik
Copy link

The filename is merge py when it should be merge.py. I submitted a PR to your repo here: Zvitorep#1 :)

@Zvitorep
Copy link
Author

The filename is merge py when it should be merge.py. I submitted a PR to your repo here: Zvitorep#1 :)

Oops! I didn't even notice that, thanks for pointing it out!

(Late night coding lol)

Fixed Python Script Filename
@FireDragon91245
Copy link

And now just merge the compilers we ned a compiler that runs in python and javascript :)

@Zvitorep
Copy link
Author

And now just merge the compilers we ned a compiler that runs in python and javascript :)

Already done, check the additions branch of my fork

@Jontes-Tech
Copy link

Beat me to it! My PR has a very similar implementation, but just does this in what I call bin/merge

1 // (lambda: exec(open("bin/merge.py").read(), globals()) or 1)()
lambda: eval("(function(){const fs=require('node:fs');const data=fs.readFileSync('bin/merge.js','utf8');return eval(data)})();")
1 // (lambda: exec(open("bin/merge.py").read(), globals()) or 1)()
lambda: eval("(function(){const fs=require('node:fs');const data=fs.readFileSync('bin/merge.js','utf8');return eval(data)})();")

@Zvitorep
Copy link
Author

Wrote a rather terrible version of the polycompiler with base64 support and "compression" over on https://github.com/Urufusan/polycompiler/tree/compression, does work though.

@lennyerik
Copy link

And now just merge the compilers we ned a compiler that runs in python and javascript :)

Already had a PR for that earlier #7 :P

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.

4 participants