docs: update hyperlinks to point to evm-monorepo#1460
Conversation
WalkthroughThis pull request updates README files in four packages (airdrops, flow, lockup, utils). Documentation links for security, contributing, and license files were changed from package-local paths to parent paths (e.g., ./SECURITY.md → ../SECURITY.md, ./CONTRIBUTING.md → ../CONTRIBUTING.md, ./LICENSE.md → ../LICENSE.md). Issue/discussion/PR links and GitHub Actions badges were updated to point to the centralized evm-monorepo and its workflows. The flow README also replaces an external Lockup URL with a relative ../lockup/ path. No exported or public declarations were modified. Suggested reviewers
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
airdrops/README.md (1)
88-89: Consider updating codecov badge to reflect monorepo structure.The codecov badge still references
sablier-labs/airdrops, while the GitHub Actions badge now points tosablier-labs/evm-monorepo. If codecov tracking has also moved to the monorepo, these badges should be consistent.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@airdrops/README.md` around lines 88 - 89, Update the codecov badge references in README by replacing the repository path "sablier-labs/airdrops" with the monorepo path "sablier-labs/evm-monorepo" for both the [codecov] and [codecov-badge] URL tokens so they match the GitHub Actions badge and current Codecov tracking.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@utils/README.md`:
- Around line 90-91: The badge reference [gha-badge] in README.md points to the
wrong repository; update the URL value currently using
"sablier-labs/evm-monorepo/actions/workflows/ci-utils.yml/badge.svg" to use this
repository's path instead (replace the owner/repo segment with the current repo
name) so the ci-utils.yml badge correctly reflects this repository's workflow
status.
---
Nitpick comments:
In `@airdrops/README.md`:
- Around line 88-89: Update the codecov badge references in README by replacing
the repository path "sablier-labs/airdrops" with the monorepo path
"sablier-labs/evm-monorepo" for both the [codecov] and [codecov-badge] URL
tokens so they match the GitHub Actions badge and current Codecov tracking.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 01baad7b-616e-4c70-af0c-e8e6f11fb1b2
📒 Files selected for processing (4)
airdrops/README.mdflow/README.mdlockup/README.mdutils/README.md
| [gha]: https://github.com/sablier-labs/evm-monorepo/actions | ||
| [gha-badge]: https://github.com/sablier-labs/evm-monorepo/actions/workflows/ci-utils.yml/badge.svg |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Verify ci-utils.yml workflow exists
fd -t f "ci-utils.yml" .github/workflows/Repository: sablier-labs/lockup
Length of output: 93
Update badge URL to reference correct repository. The workflow file ci-utils.yml exists, but the badge URL points to sablier-labs/evm-monorepo while the file is in the current repository. Update the URL to reference the correct repository.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@utils/README.md` around lines 90 - 91, The badge reference [gha-badge] in
README.md points to the wrong repository; update the URL value currently using
"sablier-labs/evm-monorepo/actions/workflows/ci-utils.yml/badge.svg" to use this
repository's path instead (replace the owner/repo segment with the current repo
name) so the ci-utils.yml badge correctly reflects this repository's workflow
status.
|
Why not raise a PR directly with |
like this issue: #1459
these were outdated and need to be merged after we rename the repo