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

Update README.md #1005

Merged
merged 1 commit into from
Nov 7, 2024
Merged

Update README.md #1005

merged 1 commit into from
Nov 7, 2024

Conversation

codelion
Copy link
Member

@codelion codelion commented Nov 7, 2024

Update URL to pepy.

Update URL to pepy.
@codelion codelion requested a review from CTY-git November 7, 2024 02:36
@patched-admin
Copy link
Contributor

The pull request review addresses a modification in the README.md file where the URL path for the pepy.tech badge link has been corrected from /project/ to /projects/. This change is purely related to documentation and does not pose any risk of bugs or security vulnerabilities, as it only updates a markdown file. The altered URL should be checked to confirm it leads to the appropriate project analytics page, ensuring users are directed correctly. The update is consistent with existing coding standards, maintaining the formatting used previously for badge links.


  • File changed: README.md
    The modification made in the README.md file has corrected the URL path from /project/ to /projects/ for the pepy.tech badge link. This change does not introduce potential bugs or security vulnerabilities, as it only affects a markdown file used for documentation purposes. However, it's important to verify that the new path indeed leads to the correct page for the project analytics to ensure users are directed appropriately.

The original coding standards are adhered to since this change is consistent with the prior format used for badge links in the markdown file.

Copy link
Contributor

@CTY-git CTY-git left a comment

Choose a reason for hiding this comment

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

LGTM

@codelion codelion closed this Nov 7, 2024
@codelion codelion reopened this Nov 7, 2024
@codelion codelion merged commit 4afe308 into main Nov 7, 2024
5 checks passed
@codelion codelion deleted the codelion-patch-1 branch November 7, 2024 03:02
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.

3 participants