-
Notifications
You must be signed in to change notification settings - Fork 3
NH-88760: add markdown lint and link #142
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great idea @xuan-cao-swi! Left a couple comments.
!lambda/.aws-sam/** | ||
!.github/pull_request_template.md | ||
!.github/ISSUE_TEMPLATE/bug-or-feature-request.md | ||
continue-on-error: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seeing this warning:
Warning: Unexpected input(s) 'continue-on-error', valid inputs are ['config', 'fix', 'globs', 'separator']
Run DavidAnson/markdownlint-cli2-action@v[1](https://github.com/solarwinds/apm-ruby/actions/runs/10395387675/job/28787280482?pr=142#step:3:1)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated.
yardoc_frontpage.md
Outdated
@@ -1,24 +1,24 @@ | |||
# Welcome to the SolarWindsAPM Ruby Gem | |||
|
|||
# Welcome to the SolarWindsAPM Ruby Gem |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Curious, how/where is this yardoc page used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yardoc page is used in https://www.rubydoc.info/gems/solarwinds_apm, but I think it's broken. Will fix it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Although, the file yardoc_frontpage.md is used in .yardopts (point yardoc_frontpage.md as default readme), but since appoptics, rubydoc.info doesn't use the yardoc_frontpage.md as default readme.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you're finding that it's not used anymore feel free to delete :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need some sort of welcome page for rubydoc.info? or the current README is sufficient?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is our current https://rubydoc.info/github/solarwinds/apm-ruby page you're referring to, yes? IMHO it is enough so we don't need the yardoc_frontpage content.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @xuan-cao-swi!
Description
Test (if applicable)