Skip to content

Comments

Docs/add documenter citations#484

Open
hkcc375 wants to merge 5 commits intoJuliaGraphs:masterfrom
hkcc375:docs/add-documenter-citations
Open

Docs/add documenter citations#484
hkcc375 wants to merge 5 commits intoJuliaGraphs:masterfrom
hkcc375:docs/add-documenter-citations

Conversation

@hkcc375
Copy link

@hkcc375 hkcc375 commented Jan 18, 2026

In this PR, I have used DocumenterCitations for citing sources of different algorithms used in Graphs.jl package. Currently, I have made changes only to centrality algorithm API. Once these changes look good, I will incorporate the changes to other files as well.

Copy link
Member

@Krastanov Krastanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for looking into that, the help is really appreciated! I left a few requests for changes.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this file? I would prefer to stick to the default unless there are specific reasons to avoid it.

Comment on lines +8 to +9
Documenter = "e30172f5-a6a5-5a46-863b-614d45cd2de4"
DocumenterCitations = "daee34ce-89f3-4625-b898-19384cb65244"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these should not be dependencies of the project as a whole, only of the docs

Comment on lines +28 to +31
## References

```@bibliography
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably be a more central page, as it will contain all references ever added, not just the references about centrality. I would suggest making a new page dedicated just to references.

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.

3 participants