Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
github-actions
bot
added
docs
|
Warning Rate limit exceeded@yamcodes has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 23 minutes and 46 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (8)
WalkthroughReplaced arkenv.vercel.app links with arkenv.js.org across READMEs and package.json metadata. No code, API, or runtime changes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
✨ Finishing Touches🧪 Generate unit tests
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
examples/basic/README.md (1)
51-51: LGTM; optional consistency tweakConsider updating the “Next steps → ArkEnv docs” link (Line 77) to the docsite for consistency: https://arkenv.js.org.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
README.md(2 hunks)apps/docs/README.md(1 hunks)apps/docs/content/docs/README.md(1 hunks)examples/basic/README.md(1 hunks)examples/with-bun/README.md(1 hunks)packages/arkenv/package.json(1 hunks)packages/vite-plugin/README.md(1 hunks)packages/vite-plugin/package.json(1 hunks)
🧰 Additional context used
🧬 Code graph analysis (2)
packages/vite-plugin/package.json (2)
packages/vite-plugin/src/index.test.ts (1)
buildWithPlugin(25-37)packages/vite-plugin/src/index.ts (1)
config(17-19)
packages/vite-plugin/README.md (2)
packages/vite-plugin/src/index.ts (1)
config(17-19)packages/vite-plugin/src/index.test.ts (1)
it(121-174)
🔇 Additional comments (7)
README.md (1)
2-2: LGTM: domain switch to js.orgAnchor updated correctly to https://arkenv.js.org.
apps/docs/README.md (1)
5-5: LGTM: live-site link updatedCopy reads well; URL points to the new domain.
packages/vite-plugin/README.md (1)
83-83: LGTM: docs link and “homepage” field updated; ready for next releaseapps/docs/content/docs/README.md (1)
5-5: LGTM: docsite URL switchedConsistent with the domain migration.
packages/vite-plugin/package.json (1)
35-35: Hold merge until JS.org DNS is live
rgshows no lingeringarkenv.vercel.applinks.- Homepage now correctly set on
arkenvand@arkenv/vite-plugin.- Once the js.org PR/DNS is live, cut a patch release so npm picks up the new homepage.
- Optional: update the GitHub repo description URL after merge.
examples/with-bun/README.md (1)
58-58: Confirm anchor availability once DNS is live. Examples referencehttps://arkenv.js.org/docs/guides/environment-configuration#using-env-files(examples/with-bun/README.md:58, examples/basic/README.md:51), but the site currently redirects/fails—please verify the#using-env-filesanchor resolves correctly after DNS and certs are active.packages/arkenv/package.json (1)
34-34: LGTM—ready to publish once arkenv.js.org is live. After the domain goes live, release a patch to update npm metadata to point to the new homepage.
@coderabbitai implement the nitpick comment |
|
Tip For best results, initiate chat on the files or code changes. 🧩 Analysis chain@yamcodes I'll implement the nitpick comment by examining the current content and updating the link on Line 77. Let me also get the full context around that line: And let me search for any "Next steps" or "ArkEnv docs" references: 🏁 Script executed: Length of output: 52 🏁 Script executed: Length of output: 330 🏁 Script executed: Length of output: 130 Perfect! I found the line that needs to be updated. On Line 77 of Current Line 77: - [ArkEnv docs](https://github.com/yamcodes/arkenv)Updated Line 77: - [ArkEnv docs](https://arkenv.js.org)This change makes the link consistent with the PR's objective of updating URLs to use the new |
… site URL at arkenv.js.org - Changed the workspace name for the www app to "www (arkenv.js.org)". - Updated documentation links in README files to point to the new site URL: https://arkenv.js.org/docs. These changes ensure consistency across the project following the recent site migration.
- Consolidated example entries in the workspace configuration for clarity. - Ensured consistent formatting and structure across all example paths. These changes enhance the organization of the workspace, making it easier to navigate and manage examples.
1 participant
Once js-org/js.org#10128 resolves:
Summary by CodeRabbit