-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
retain postfix node in type section typed AST #23096
Conversation
PR to unchained: SciNim/Unchained#45 I think we should disable NESM indefinitely. We've been maintaining a fork for a year and I don't want to sift through 3000 lines of test output to adapt it. The failing test also seems like it directly matches against tree output of AST, which would now become dependent on the Nim version. |
Thanks for your hard work on this PR! Hint: mm: orc; opt: speed; options: -d:release |
Sorry, I skipped CI for the last commit, it wasn't actually passing yet. Should be fine after SciNim/Unchained#45 is merged though. |
Never mind, some doc tests fail too. |
Continued from #23096 which was reverted due to breaking a package and failing docgen tests. Docgen should now work, but ~~a PR is still pending for the package: SciNim/Unchained#45 has been merged
fixes #22933