Skip to content

Conversation

@brkn
Copy link
Contributor

@brkn brkn commented Dec 1, 2025

Helper methods link is broken at master branch

To reproduce

  1. go to StringScanner docs
  2. Click to link at line

    See examples at helper_methods

  3. Resolved url gives 404: https://docs.ruby-lang.org/en/master/strscan/helper_methods_md.html

Fix

Currently link resolves as href="doc/strscan/helper_methods_md.html"
Correct link should be resolved as href="helper_methods_md.html"

@kou kou changed the title Docs - Fix broken link to helper methods [DOC] Fix broken link to helper methods Dec 1, 2025
@kou kou merged commit adb8678 into ruby:master Dec 1, 2025
37 of 38 checks passed
@kou
Copy link
Member

kou commented Dec 1, 2025

Good catch!

matzbot pushed a commit to ruby/ruby that referenced this pull request Dec 1, 2025
(ruby/strscan#179)

### Helper methods link is broken at master branch

To reproduce
1. go to [StringScanner
docs](https://docs.ruby-lang.org/en/master/StringScanner.html)
2. Click to link at line
   > See examples at **helper_methods**
3. Resolved url gives 404:
https://docs.ruby-lang.org/en/master/strscan/helper_methods_md.html

### Fix

Currently link resolves as `href="doc/strscan/helper_methods_md.html"`
Correct link should be resolved as `href="helper_methods_md.html"`

ruby/strscan@adb8678aa6
tagomoris pushed a commit to tagomoris/ruby that referenced this pull request Dec 2, 2025
(ruby/strscan#179)

### Helper methods link is broken at master branch

To reproduce
1. go to [StringScanner
docs](https://docs.ruby-lang.org/en/master/StringScanner.html)
2. Click to link at line
   > See examples at **helper_methods**
3. Resolved url gives 404:
https://docs.ruby-lang.org/en/master/strscan/helper_methods_md.html

### Fix

Currently link resolves as `href="doc/strscan/helper_methods_md.html"`
Correct link should be resolved as `href="helper_methods_md.html"`

ruby/strscan@adb8678aa6
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