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

Include a link to documentation page in project URLs #766

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

mslw
Copy link
Collaborator

@mslw mslw commented Oct 17, 2024

The project.urls are picked up and displayed (with matching icons, if
possible) by pypi.org. DataLad-next has its online documentation under
docs.datalad.org and it is pretty good. Given that we already include
a GitHub link in project.urls anyway (twice, in fact), it could be
better to point Documentation to the online docs, rather than the
GitHub README.

What do you think?

Side note: I would even be tempted to remove the Homepage link and
only keep Source (currently they are the same and point to the GitHub
project), but I wasn't sure if the Homepage is considered necessary or
not, and hence didn't include it in the original proposal.

The project.urls are picked up and displayed (with matching icons, if
possible) by pypi.org. DataLad-next has its online documentation under
docs.datalad.org and it is pretty good. Given that we already include
a GitHub link in project.urls anyway (twice, in fact), it could be
better to point Documentation to the online docs, rather than the
GitHub README.
@mslw mslw requested a review from mih as a code owner October 17, 2024 15:54
Copy link
Member

@mih mih left a comment

Choose a reason for hiding this comment

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

Makes sense. Thanks!

@mih mih merged commit 39a9d83 into datalad:main Oct 17, 2024
5 checks passed
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