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

Add support for capital-case Justfile #17

Closed
wants to merge 1 commit into from

Conversation

mihaigalos
Copy link

Currently, capital-case Justfile files are not syntax-highlighted.
This attempts to address that.

@mihaigalos
Copy link
Author

Would like to create a test for this, but don't know how to best approach it.
Please advise.

@NoahTheDuke
Copy link
Owner

I have no earthly idea how we'd test it, hah.

@mihaigalos
Copy link
Author

mihaigalos commented Oct 10, 2021

Maybe you know how to instruct a package manager to consume this branch/PR when installing?
Or installing it locally in .vim, check highlighting directly using vim.

@NoahTheDuke
Copy link
Owner

Oh I thought you meant using the hand-built automated test system. Manually testing is as easy as pointing the Plug command at the local directory instead of the git repo url

@mihaigalos
Copy link
Author

Wait, is e23d9ec939fcc3560ce7983bd9e532441fbf47a0, Aug 27 not already treating case-insensitive just files?
I got my language support from vim-polyglot, which apparently contains the just.vim, Jun 9 from before that commit.

I'm confused, can you have a look? I guess I can close this now.

@NoahTheDuke
Copy link
Owner

I'll take a look! Maybe I need to cut a new release so polyglot picks it up

@mihaigalos
Copy link
Author

Ok. In the meantime, I've contributed a PR to sheerun/vim-polyglot#756.

@mihaigalos mihaigalos closed this Oct 12, 2021
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