-
Notifications
You must be signed in to change notification settings - Fork 11
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
Adapt to https://github.com/coq/coq/pull/19530 #15
base: master
Are you sure you want to change the base?
Conversation
As mentioned in #10 (comment), changes to the textbook first get merged into a private source repo, and then compiles to here. Please acknowledge the following copyright transfer, and send an email to @bcpierce00. Lines 452 to 491 in 43b800e
|
@liyishuai the policy of Coq CI is that the repo/branch in there should accept overlay PRs. Do you confirm that such PRs should instead go to another repo (which one?), in that case, we should at the very least mention this in https://github.com/coq/coq/blob/718eabc3e8a6ad877ca875fa6647b931b5ffadd4/dev/ci/ci-basic-overlay.sh#L456-L457 |
Yes, this repo is a compilation result, rather than a "source code". @bcpierce00 Do you feel like handling this PR and editing the documentation for Coq CI? |
Adapt to coq/coq#19530
This is an adaptation in anticipation of the day the temporary backward compatibility introduced in the upstream PR will be removed (probably a few years in the future).
Merging this is not required for the upstream PR, you can do whatever you want with it.