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

jack implementation #191

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

Conversation

victoriaprisco
Copy link

nand2tetris!

@mawerty
Copy link
Owner

mawerty commented Jan 12, 2024

Hey, you forgot to make normal implementation and just made optimised implementation

@victoriaprisco
Copy link
Author

oops sorry. fixed now (maybe)

@mawerty
Copy link
Owner

mawerty commented Jan 12, 2024

One more question, you have also modified .pl files, is this intended and should I merge it?

@victoriaprisco
Copy link
Author

victoriaprisco commented Jan 12, 2024

totally not intentional. sorry about that (not meant to be merged)

@mawerty
Copy link
Owner

mawerty commented Jan 12, 2024

Fuck, github desktop is having some problems with main.pl and Main.pl files, he thinks that they are this same or something and he is trying to overrite one with another or some shit like this.

@mawerty
Copy link
Owner

mawerty commented Jan 12, 2024

Tommorow, when I will have more free time, I will fix this and merge the code

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