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

deprecate Cd command #16119

Closed
wants to merge 2 commits into from
Closed

deprecate Cd command #16119

wants to merge 2 commits into from

Conversation

Alizter
Copy link
Contributor

@Alizter Alizter commented May 31, 2022

Keeping this as a draft for now. We need to provide output directory support for extraction first.

@Alizter Alizter added the kind: cleanup Code removal, deprecation, refactorings, etc. label May 31, 2022
@Alizter
Copy link
Contributor Author

Alizter commented Feb 29, 2024

@herbelin @SkySkimmer There was this plan to deprecate Cd entirely because it was only used in extraction. It might help with the output directory PR. If you disallow output directory and Cd to be used, it might simplofy the implementation.

We can deprecate Cd as extraction can now have its output directory set IIRC. That was the only blocker.

@Alizter
Copy link
Contributor Author

Alizter commented Feb 29, 2024

Oh I see that you've already seen this so nevermind then.

@herbelin
Copy link
Member

herbelin commented Mar 4, 2024

I realize that I apparently misinterpreted this PR as an issue and opened #17403 which at the end is redundant. I can close #17403 so that you can reopen #16119.

@Alizter
Copy link
Contributor Author

Alizter commented Mar 4, 2024

@herbelin I won't have time to work on this so feel free to take over. I just wanted to make sure you knew about it.

herbelin added a commit to herbelin/github-coq that referenced this pull request Mar 29, 2024
herbelin added a commit to herbelin/github-coq that referenced this pull request Apr 2, 2024
herbelin added a commit to herbelin/github-coq that referenced this pull request Apr 5, 2024
herbelin added a commit to herbelin/github-coq that referenced this pull request Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: cleanup Code removal, deprecation, refactorings, etc.
Projects
Archived in project
Extraction
  
To do
Development

Successfully merging this pull request may close these issues.

None yet

2 participants