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 two references and fix one #437

Merged
merged 4 commits into from
Feb 4, 2024
Merged

add two references and fix one #437

merged 4 commits into from
Feb 4, 2024

Conversation

faenuccio
Copy link
Contributor

Fix one ref to an ITP paper and add two more

@faenuccio faenuccio changed the title add three refs add three references Feb 4, 2024
@faenuccio faenuccio changed the title add three references add two references and fix one Feb 4, 2024
@robertylewis robertylewis merged commit 6995221 into leanprover-community:lean4 Feb 4, 2024
2 checks passed
@faenuccio faenuccio deleted the fae_added_three_refs branch February 4, 2024 18:52
@robertylewis
Copy link
Member

I'm confused about why CI succeeded on this PR -- the build on the main branch is failing because of dulicated references. @faenuccio once the site has built and deployed, could you check that all the references you expect are being displayed at https://leanprover-community.github.io/papers.html ?

@faenuccio
Copy link
Contributor Author

Sure, I will keep on eye and will report as soon as I see that the site has built.

@faenuccio
Copy link
Contributor Author

OK, I have double-checked and there is no duplicate. There was one problem with a curly bracket (fixed in #438 ) but I think that the duplicate warning comes from the fact that there are two submissions with almost the same name (but for capitalization) that are actually two different papers.

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