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

Back to the future #49

Closed
wants to merge 5 commits into from
Closed

Back to the future #49

wants to merge 5 commits into from

Conversation

mbovel
Copy link
Contributor

@mbovel mbovel commented Jul 9, 2020

As discussed: errors are perhaps better on AST nodes directly. And also, I rewrote almost everything. I know some documentation is still missing. Feel free to ping for any question about anything.

@mbovel mbovel requested a review from MaximeKjaer July 9, 2020 22:08
@mbovel mbovel self-assigned this Jul 9, 2020
@mbovel
Copy link
Contributor Author

mbovel commented Jul 12, 2020

Coverage problem was due to the fact that I forgot to remove some files. New result: "Coverage increased (+1.4%) to 94.757%".

@mbovel mbovel closed this Nov 5, 2021
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.

None yet

1 participant