Skip to content

Add github URL in README#135

Merged
pks-t merged 1 commit intoclar-test:mainfrom
sthibaul:main
Mar 17, 2026
Merged

Add github URL in README#135
pks-t merged 1 commit intoclar-test:mainfrom
sthibaul:main

Conversation

@sthibaul
Copy link
Copy Markdown
Contributor

So that imports into other projects (e.g. git) link to the upstream repository.

Copy link
Copy Markdown

@pks-gitlab pks-gitlab left a comment

Choose a reason for hiding this comment

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

@sthibaul I think it makes sense to provide the link, but I think it would make sense to integrate it into the prose a bit better. Thanks!

@sthibaul sthibaul force-pushed the main branch 3 times, most recently from cdaba06 to 4251949 Compare March 16, 2026 10:25
README.md Outdated
currently maintained by [Edward Thomson](https://github.com/ethomson),
and used by the [libgit2][libgit2] and [git][git] projects, amongst
others.
currently maintained on [github][github] by
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

One more nit:

Suggested change
currently maintained on [github][github] by
currently maintained on [GitHub][github] by

Other than that this reads good to me, thanks!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done so

So that imports into other projects (e.g. git) link to the upstream
repository.
@pks-t
Copy link
Copy Markdown
Member

pks-t commented Mar 17, 2026

Thanks, looks good to me now!

@pks-t pks-t merged commit d83036a into clar-test:main Mar 17, 2026
7 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.

3 participants