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

Deduplicate field logic into runtime utility #14

Closed
shadaj opened this issue Dec 30, 2022 · 1 comment
Closed

Deduplicate field logic into runtime utility #14

shadaj opened this issue Dec 30, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@shadaj
Copy link
Member

shadaj commented Dec 30, 2022

To reduce the size of generated macro code

@shadaj shadaj added the enhancement New feature or request label Dec 30, 2022
ilonachan pushed a commit to ilonachan/rust-sitter that referenced this issue Sep 24, 2023
ilonachan pushed a commit to ilonachan/rust-sitter that referenced this issue Sep 24, 2023
ilonachan pushed a commit to ilonachan/rust-sitter that referenced this issue Oct 16, 2023
shadaj pushed a commit that referenced this issue Nov 4, 2023
* Deduplicate field logic into runtime utility #14

* output of insta snapshot tests changed intentionally

* moved macro functions to __private subpackage with warning

* lint

* more lint

* improved generated output, moved parse to runtime

---------

Co-authored-by: ilonachan <[email protected]>
@shadaj
Copy link
Member Author

shadaj commented Nov 5, 2023

Fixed by #47

@shadaj shadaj closed this as completed Nov 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant