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

Tests to verify JavaScript, (some) TypeScript, and Rust calculations #15

Merged
merged 5 commits into from
Apr 13, 2024

Conversation

themkat
Copy link
Contributor

@themkat themkat commented Apr 13, 2024

Like the title says. The TypeScript tests are minimal, as most of our support already builds on JavaScript. Might add more later, but didn't really feel motivating as it would be pretty much the same as JavaScript. JavaScript and Rust tests are more complete, testing most of the rules.

Also fix a minor issue with anonymous functions not working properly due to wrong node name in JavaScript.

@jcs090218
Copy link
Member

Thank you so much! 🚀

Let me know if you are willing to help maintain this package since you have recently sent multiple high-quality patches. :)

@jcs090218 jcs090218 merged commit da9b300 into emacs-vs:master Apr 13, 2024
11 of 12 checks passed
@themkat themkat deleted the javascript_rust_tests branch April 14, 2024 12:21
@themkat
Copy link
Contributor Author

themkat commented Apr 14, 2024

Thank you so much! 🚀

Let me know if you are willing to help maintain this package since you have recently sent multiple high-quality patches. :)

Sure 🙂 I would love to help out with maintaining this package! 😄

@jcs090218
Copy link
Member

Awesome! I've invited you to the group! 🥳

I have the branch protection rules set up first, so you must still have at least one approval before merging. Once you are fully qualified, I will give you full control over the repo. :D

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