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

Update CITATION.cff #46

Merged
merged 1 commit into from
Apr 18, 2023
Merged

Update CITATION.cff #46

merged 1 commit into from
Apr 18, 2023

Conversation

tlubowe
Copy link
Collaborator

@tlubowe tlubowe commented Apr 17, 2023

We want people to know this is built by NVIDIA in the papers.

We want people to know this is built by NVIDIA in the papers.
@leofang
Copy link
Member

leofang commented Apr 17, 2023

FYI previously (before 23.03) the auto-generated title is "NVIDIA/cuQuantum: cuQuantum vX.Y.Z". I don't have any opinion in this, so feel free to merge Tom! 🙂

@tlubowe
Copy link
Collaborator Author

tlubowe commented Apr 17, 2023

Ah yes, that is ideal, I was just adding this now as I had noticed that it doesnt appear if you click the cite the repo dropdown and view the ADA format but with this change that will be clear.

I can't seem to merge as I need 1 approving review, but PR authors cant approve their own PRs, just comment (unless I am looking in the wrong place).

@leofang
Copy link
Member

leofang commented Apr 18, 2023

Yeah, I think the security setting was updated lately. Merging!

@leofang leofang merged commit 2cc68ed into main Apr 18, 2023
@leofang leofang deleted the tlubowe-patch-1 branch April 18, 2023 00:06
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