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

[serlib] Remove serlib and use coq-lsp serlib version instead. #409

Merged
merged 2 commits into from
Jun 4, 2024

Conversation

ejgallego
Copy link
Owner

@ejgallego ejgallego commented May 9, 2024

@ejgallego ejgallego force-pushed the serlib_merge branch 8 times, most recently from 598c5c9 to 75d0835 Compare May 9, 2024 16:32
@ejgallego ejgallego added this to the 0.20.0 milestone Jun 3, 2024
@ejgallego ejgallego marked this pull request as ready for review June 3, 2024 16:56
ejgallego added a commit to ejgallego/coq that referenced this pull request Jun 3, 2024
Now serapi depends on coq_lsp, also serapi tests have been moved
there.

To coordinate with the merge of
ejgallego/coq-serapi#409
ejgallego added a commit to ejgallego/coq that referenced this pull request Jun 3, 2024
Now serapi depends on coq_lsp, also serapi tests have been moved
there.

To coordinate with the merge of
ejgallego/coq-serapi#409
@ejgallego ejgallego force-pushed the serlib_merge branch 3 times, most recently from 2ef54ad to 4d3b175 Compare June 3, 2024 18:12
ejgallego added a commit to ejgallego/coq that referenced this pull request Jun 3, 2024
Now serapi depends on coq_lsp, also serapi tests have been moved
there.

To coordinate with the merge of
ejgallego/coq-serapi#409
ejgallego added a commit to ejgallego/coq that referenced this pull request Jun 4, 2024
Now serapi depends on coq_lsp, also serapi tests have been moved
there.

To coordinate with the merge of
ejgallego/coq-serapi#409
This is the right choice as serlib is now hosted there.
@ejgallego ejgallego merged commit e59d1dc into main Jun 4, 2024
4 checks passed
@ejgallego ejgallego deleted the serlib_merge branch June 4, 2024 15:38
ejgallego added a commit to ejgallego/coq that referenced this pull request Jun 5, 2024
Now serapi depends on coq_lsp, also serapi tests have been moved
there.

To coordinate with the merge of
ejgallego/coq-serapi#409
@ejgallego ejgallego restored the serlib_merge branch June 5, 2024 11:12
@ejgallego ejgallego deleted the serlib_merge branch June 5, 2024 11:13
@ejgallego ejgallego restored the serlib_merge branch June 5, 2024 11:13
ejgallego added a commit to ejgallego/coq that referenced this pull request Jun 5, 2024
Now serapi depends on coq_lsp, also serapi tests have been moved
there.

To coordinate with the merge of
ejgallego/coq-serapi#409
@ejgallego ejgallego deleted the serlib_merge branch June 5, 2024 16:15
@ejgallego ejgallego mentioned this pull request Jun 5, 2024
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.

None yet

1 participant