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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set metadata source_code_uri #1669

Closed
wants to merge 1 commit into from
Closed

Set metadata source_code_uri #1669

wants to merge 1 commit into from

Conversation

sato11
Copy link
Contributor

@sato11 sato11 commented Nov 30, 2023

This commit adds 'source_code_uri' key to gemspec's metadata, which makes nanoc and the plugins compatible with gem-src's functionality: without it there is no way to detect the source repo unless actually opening the homepage and navigate through it, which I believe is rather tedious.

The 'source_code_uri' item in gemspec's metadata is described in the guideline rubygems.org publishes, and it is what gem-src uses to prioritize when inferring which repo to clone.

I guess there are not many who have come across this issue, but what do you think of it anyway?

And last but not least, thanks for letting me have a chance to touch your product. I have had a lot of fun and I appreciate your effort 馃

This commit adds 'source_code_uri' key to gemspec's metadata,
which makes nanoc and the plugins compatible with
[gem-src](https://rubygems.org/gems/gem-src)'s functionality:
without it there is no way to detect the source repo
unless actually opening the homepage and navigate through it,
which I believe is rather tedious.

The 'source_code_uri' item in gemspec's metadata is described in
[the guideline rubygems.org publishes](https://guides.rubygems.org/specification-reference/#metadata),
and it is
[what gem-src uses to prioritize when inferring which repo to clone](https://github.com/amatsuda/gem-src/blob/v0.10.2/lib/rubygems_plugin.rb#L38).
@denisdefreyne
Copy link
Member

@sato11 Thank you!

I made two changes:

  1. Fix Rubocop style (rubocop -a)

  2. Fix the URL for nanoc. This one is tagged with just the version number, not as nanoc-v#{VERSION} for historical reasons.

In any case, the change is merged!

@denisdefreyne
Copy link
Member

This is released in Nanoc 4.12.19.

@sato11 sato11 deleted the source-code-uri branch December 12, 2023 01:21
@sato11
Copy link
Contributor Author

sato11 commented Dec 12, 2023

@denisdefreyne Thanks for your passionate work!

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.

None yet

2 participants