Skip to content

Conversation

SkySkimmer
Copy link
Collaborator

No description provided.

@ejgallego
Copy link
Owner

@SkySkimmer coq-lsp CI setup uses pinned commits for tightly coupled dependencies.

You are most welcome to bump the submodule id's if desired, make submodule-updates usually does the trick.

@SkySkimmer
Copy link
Collaborator Author

This change may be backwards compatible (the removed type hasn't been used in the parser produced ast for a while AFAIK), let's see if rerunning CI after #950 works

@SkySkimmer SkySkimmer closed this May 6, 2025
@SkySkimmer SkySkimmer reopened this May 6, 2025
@ejgallego
Copy link
Owner

Indeed, change looks good to me!

@ejgallego ejgallego marked this pull request as ready for review May 6, 2025 15:41
@ejgallego ejgallego merged commit fbd949c into ejgallego:main May 6, 2025
14 of 30 checks passed
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