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

Add source code and changelog links to gemspec #318

Merged
merged 1 commit into from
Oct 30, 2017
Merged

Add source code and changelog links to gemspec #318

merged 1 commit into from
Oct 30, 2017

Conversation

greysteil
Copy link
Contributor

Makes it easy to programatically find the source code and changelog for rack-mini-profiler, using the rubygems API. More details on the direction of travel for gemspecs is here and here, and the current state is here.

(My interest in this is so that Dependabot can find a link to the rack-mini-profiler source code when it creates PRs.)

@codecov-io
Copy link

codecov-io commented Oct 30, 2017

Codecov Report

Merging #318 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #318   +/-   ##
=======================================
  Coverage   85.44%   85.44%           
=======================================
  Files          21       21           
  Lines        1209     1209           
=======================================
  Hits         1033     1033           
  Misses        176      176

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3cdae97...2201965. Read the comment docs.

@SamSaffron
Copy link
Member

looks good!

@SamSaffron SamSaffron merged commit 5c759bd into MiniProfiler:master Oct 30, 2017
@greysteil greysteil deleted the source-code-link branch October 30, 2017 09:05
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