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

Link to github, not rubyforge [ci skip] #2121

Merged
merged 1 commit into from
Jul 10, 2023
Merged

Conversation

olleolleolle
Copy link
Contributor

Overview

Updates a link to defunct website

Submitter Checklist:

  • Include a link to the related GitHub issue, if applicable
  • Include a security review link, if applicable

Testing

Q: Would it be possible to add some "link checker" build step? Or have such a manual task, every year?

Reviewer Checklist

  • Perform code review
  • Add performance label
  • Perform appropriate level of performance testing
  • Confirm all checks passed
  • Add version label prior to acceptance

@CLAassistant
Copy link

CLAassistant commented Jul 10, 2023

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added the community To tag external issues and PRs submitted by the community label Jul 10, 2023
@fallwith
Copy link
Contributor

Much appreciated, @olleolleolle, thank you!

@fallwith fallwith merged commit da97278 into newrelic:dev Jul 10, 2023
@olleolleolle olleolleolle deleted the patch-1 branch July 10, 2023 20:53
@fallwith
Copy link
Contributor

Q: Would it be possible to add some "link checker" build step? Or have such a manual task, every year?

Hi @olleolleolle, thanks very much for this wonderful suggestion! With #2127, we will now evaluate URL availability health for all links in source code comments. The check happens once a day. It managed to find fully 5 additional outdated links. We very much appreciate your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community To tag external issues and PRs submitted by the community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants