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

Add some snippets #47

Merged
merged 3 commits into from
Jul 20, 2021
Merged

Add some snippets #47

merged 3 commits into from
Jul 20, 2021

Conversation

Zimmi48
Copy link
Member

@Zimmi48 Zimmi48 commented Jul 20, 2021

I did some experiments locally. However, for some reason, I couldn't yet produce a PDF with the Coq answers, so I will use this PR to check if that's produced correctly.

@Casteran
Copy link
Member

Casteran commented Jul 20, 2021 via email

@Zimmi48
Copy link
Member Author

Zimmi48 commented Jul 20, 2021

Everything, but that's because my coq-serapi is not correctly installed. The problem doesn't show up in the version built by the CI.

@Casteran
Copy link
Member

Ok, so it's safe to merge this PR to master now ?

@Zimmi48
Copy link
Member Author

Zimmi48 commented Jul 20, 2021

The Check were printing output contrary to the previous version so I've pushed a new commit to try to mask the output. I'm waiting for the result of CI to see if that works.

@Zimmi48
Copy link
Member Author

Zimmi48 commented Jul 20, 2021

OK, it worked! So this is ready to merge.

@Casteran Casteran merged commit 6e4a62e into coq-community:master Jul 20, 2021
@Zimmi48 Zimmi48 deleted the add-some-snippets branch July 20, 2021 11:36
@Casteran
Copy link
Member

How about adding in the pdf doc and README.md a paragraph like:
" An increasing part of the hydras.pdf document is written with the help of the Alectryon software [...], replacing old
copy-pasted quotations with snippets generated from the source and answers by the system.
<descrition of the snippet tools, and credits>. "

@Zimmi48
Copy link
Member Author

Zimmi48 commented Jul 20, 2021

Good idea

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants