-
Notifications
You must be signed in to change notification settings - Fork 39
Add Erlang support #196
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
base: main
Are you sure you want to change the base?
Add Erlang support #196
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request adds Erlang language support to the spell-checking tool by integrating the WhatsApp tree-sitter-erlang parser and including a comprehensive word list from OTP libraries.
Key changes:
- Adds tree-sitter-erlang dependency (version 0.15.0)
- Creates Erlang-specific tree-sitter query file for spell-checking
- Includes a 580-word Erlang-specific dictionary
- Provides example Erlang code and comprehensive tests
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 46 comments.
Show a summary per file
| File | Description |
|---|---|
| word_lists/erlang.txt | New Erlang-specific word list with 580 entries from OTP libraries (contains 38 duplicate entries and minor formatting issues) |
| examples/example.erl | Example Erlang file demonstrating spell-checking capabilities with intentional misspellings |
| crates/codebook/tests/test_erlang.rs | Comprehensive test suite for Erlang spell-checking functionality |
| crates/codebook/src/queries/erlang.scm | Tree-sitter query definitions for capturing Erlang comments, strings, atoms, variables, and function names |
| crates/codebook/src/queries.rs | Registers Erlang language type and settings in the spell-checker |
| crates/codebook/src/dictionaries/combined.gen.txt | Generated combined dictionary file including Erlang words |
| crates/codebook/Cargo.toml | Adds tree-sitter-erlang workspace dependency |
| Cargo.toml | Specifies tree-sitter-erlang version 0.15.0 |
| Cargo.lock | Lock file update with tree-sitter-erlang dependency |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
f953f68 to
7fff459
Compare
7fff459 to
b38d266
Compare
fffd51a to
49c2580
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 8 out of 9 changed files in this pull request and generated 18 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
49c2580 to
c3776e0
Compare
c3776e0 to
a6217f1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 8 out of 9 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
4793483 to
8f52b29
Compare
|
Ooof, copilot went off the rails on flagging spelling issues. I've pushed instructions to avoid that in the future. Anyway, thanks for putting this together! Is it ready for review? |
|
Yes. It is ready. |
Add Erlang support, use WhatsApp erlang tree-sitter. Including word list from OTP libraries.