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

Correcting typo in instructions for making tree-sitter support #6

Merged
merged 4 commits into from
Oct 7, 2023

Conversation

slackline
Copy link
Contributor

The instructions had ./src/scanner.cc on the second line (i.e. an extra c in the file extension) which I have corrected.

At the same time I've...

  • Moved the instructions from a list to code blocks and changed to sub-headings.
  • Moved comment in brackets about /path/to/tree-sitter-langs to the start of the section and included a link to the package.
  • Changed instruction for installation to be added to init.el rather than .emacs as this is the more common location (often init.el is nested within a .emacs directory).
  • Added instructions for installing and configuring with use-package.
  • Fixed the workflow status badge.

The instructions had `./src/scanner.cc` on the second line (i.e. an extra `c` in the file extension) which I have
corrected.

At the same time I've...

+ Moved the instructions from a list to code blocks and changed to sub-headings.
+ Moved comment in brackets about `/path/to/tree-sitter-langs` to the start of the section and included a link to the
package.
+ Changed instruction for installation to be added to `init.el` rather than `.emacs` as this is the more common
location (often `init.el` is nested within a `.emacs` directory).
+ Added instructions for installing and configuring with `use-package`.
@ShuguangSun ShuguangSun merged commit 9bedf49 into ShuguangSun:master Oct 7, 2023
1 check failed
@slackline
Copy link
Contributor Author

Hey @ShuguangSun thanks for accepting and merging my PR.

Would you mind tagging this with this with the label hacktoberfest-accepted as I'm participating in HactoberFest 2023 and would like this to contribute towards the total.

Noticed the CI was failing and have created an issue but am unsure how to address that as well as making another PR to add support for CI testing on Emacs 29.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants