Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix: use relative url for submodule definition (#911) #912

Merged
merged 2 commits into from
Jun 7, 2023
Merged

fix: use relative url for submodule definition (#911) #912

merged 2 commits into from
Jun 7, 2023

Conversation

gongfarmer
Copy link
Contributor

Tested within secure environment without GitHub access.
Tested in normal situation with GitHub access.

@L3MON4D3
Copy link
Owner

L3MON4D3 commented Jun 7, 2023

Ah, nice👍
I'm a tiny bit hesitant because I have no idea if luasnip and jsregexp will always be developed on the same service, but I guess until then this is pretty practical.

Thank you!

@L3MON4D3 L3MON4D3 merged commit 7a564d7 into L3MON4D3:master Jun 7, 2023
@L3MON4D3 L3MON4D3 mentioned this pull request Jun 7, 2023
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gongfarmer @L3MON4D3