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

Scanner uses operators json mm #90

Closed
wants to merge 11 commits into from
Closed

Conversation

mmatera
Copy link
Contributor

@mmatera mmatera commented Nov 25, 2024

Just to check why this is not running all the tests

rocky and others added 10 commits November 24, 2024 11:56
Function unicode change so as not to conflict with RightTeeArrow.
Function unicode is a long arrow.

Remove CSV to YML stuff. CSV is beyond hope of keeping in sync.

Remove tokeniser import from __init__.py. Workaround for now.
We need this so we can create operator JSON without needing the
JSON table to be prevously around.

clanup tokeniser.py a little bit.

SPlit typing changing variables like tokens and literal_tokens into
separate variables for each type they can hold.
@rocky
Copy link
Member

rocky commented Nov 25, 2024

@mmatera Please feel free to just take over from here with all the operator stuff.

While I am happy we've done this, or made a at least a first stab, I am tired of this and have many other things I'd rather do.

I got into this because you suggested expanding the Star[] Built-in with no predefined meaning to all of them. As I said previously, this was a bit of work to do. (And there still is — there is a bit more that could be done to clean up and improve things.)

@mmatera
Copy link
Contributor Author

mmatera commented Nov 25, 2024

@mmatera Please feel free to just take over from here with all the operator stuff.

@rocky, again, sorry for the collision. From your previous comment, I thought you wanted to continue it. I realized you had made another commit after merging.

While I am happy we've done this, or made a at least a first stab, I am tired of this and have many other things I'd rather do.

Actually I find this a huge step, also in the part I am trying to do with formatting.

I got into this because you suggested expanding the Star[] Built-in with no predefined meaning to all of them. As I said previously, this was a bit of work to do. (And there still is — there is a bit more that could be done to clean up and improve things.)

Thanks for all this work. This makes it easier to continue in this line.

@mmatera Please feel free to just take over from here with all the operator stuff.

While I am happy we've done this, or made a at least a first stab, I am tired of this and have many other things I'd rather do.

I got into this because you suggested expanding the Star[] Built-in with no predefined meaning to all of them. As I said previously, this was a bit of work to do. (And there still is — there is a bit more that could be done to clean up and improve things.)

@mmatera mmatera closed this Nov 25, 2024
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