Skip to content

Conversation

xuan-cao-swi
Copy link
Contributor

Description

Test (if applicable)

@xuan-cao-swi xuan-cao-swi requested a review from a team as a code owner August 14, 2024 21:16
@xuan-cao-swi xuan-cao-swi mentioned this pull request Aug 19, 2024
Copy link
Contributor

@cheempz cheempz left a 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
Copy link
Contributor

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)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated.

@@ -1,24 +1,24 @@
# Welcome to the SolarWindsAPM Ruby Gem

# Welcome to the SolarWindsAPM Ruby Gem
Copy link
Contributor

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?

Copy link
Contributor Author

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.

Copy link
Contributor Author

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.

Copy link
Contributor

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 :)

Copy link
Contributor Author

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?

Copy link
Contributor

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated.

Copy link
Contributor

@cheempz cheempz left a 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!

@xuan-cao-swi xuan-cao-swi merged commit 37f11c0 into main Aug 23, 2024
14 checks passed
@xuan-cao-swi xuan-cao-swi deleted the NH-88760 branch August 23, 2024 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants