Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
docs: add community health files for A+ portfolio standards #26
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
Uh oh!
There was an error while loading. Please reload this page.
docs: add community health files for A+ portfolio standards #26
Changes from all commits
96ca99dFile filter
Filter by extension
Conversations
Uh oh!
There was an error while loading. Please reload this page.
Jump to
Uh oh!
There was an error while loading. Please reload this page.
There are no files selected for viewing
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.
Replace the placeholder
vX.Y.Zwith a valid version tag or remove the comparison link until a first release is published. The current link will return a 404 error.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.
Remove Python-specific example from a Node.js/TypeScript project. This causes confusion for contributors attempting to set up the development environment.
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 repository is a Node.js/TypeScript project, but the development setup instructions contain Python-specific commands (
pip install). Please update these instructions to use Node.js commands (such asnpm installandnpm test) to match the project's actual tech stack.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.
Remove reference to CODE_OF_CONDUCT.md as this file is not included in the PR and does not exist in the repository. The broken link will confuse contributors.
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.
The repository does not currently contain a
CODE_OF_CONDUCT.mdfile, so this relative link will result in a 404 error. Please either add theCODE_OF_CONDUCT.mdfile to the repository, link to an external Code of Conduct URL, or remove/comment out this section until the file is added.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.
Since
SUPPORT.mdis located in the root directory of the repository (rather than in a.github/subdirectory), the relative links../../issuesand../../discussionswill resolve incorrectly (resulting in 404 errors on GitHub). Using root-relative paths starting with/will ensure these links resolve correctly to the repository's tabs.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.
Remove references to SECURITY.md as this file is not included in the PR and does not exist in the repository. Contributors attempting to report security vulnerabilities will encounter broken links.
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 new support path tells reporters to email security concerns via
SECURITY.md, but the repository'sSECURITY.mdcontains no email address and only says to use GitHub private vulnerability reporting "if enabled" or contact the maintainer directly. In repositories without private vulnerability reporting enabled, a reporter following these instructions is left with no concrete private disclosure channel after being told not to open a public issue.Useful? React with 👍 / 👎.
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.
🛑 PII Exposure: Replace personal name in LinkedIn URL with a placeholder to avoid exposing personally identifiable information in public documentation.
Uh oh!
There was an error while loading. Please reload this page.