docs(deployment): replace dead ethgasstation.info reference - #1292
docs(deployment): replace dead ethgasstation.info reference#1292Flotapponnier wants to merge 1 commit into
Conversation
📝 WalkthroughWalkthroughThe deployment guide replaces the gas-price reference with Etherscan Gas Tracker and OpenChainBench while retaining the 30 gwei threshold and minimum 6 ETH funding guidance. ChangesDeployment documentation
Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/deployment.md`:
- Line 188: Update the deployment instruction’s gas-price reference to include
spacing between the numeric value and unit, using “30 gwei” instead of “30gwei”;
optionally improve the surrounding conditional wording without changing its
meaning.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 5e634f6a-511a-414c-a877-254d25c21bd3
📒 Files selected for processing (1)
docs/deployment.md
Signed-off-by: Florent Tapponnier <contact@openchainbench.com>
235fde0 to
b027c4a
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/deployment.md`:
- Line 188: Update the funding instruction near the Etherscan Gas Tracker
reference by replacing the OpenChainBench link with a direct current gas-price
or fee-estimate source. Keep the existing guidance about increasing the ETH
amount when gas prices exceed 30 gwei.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: d1c2db64-6d71-4590-abc3-fa56514bdd98
📒 Files selected for processing (1)
docs/deployment.md
| If this passes successfully it will print the deployer address and the current ETH balance. Next: | ||
|
|
||
| 1. Send at least 6 ETH to this address (check ethgasstation.info; if gas prices are > 30gwei then we may need more) | ||
| 1. Send at least 6 ETH to this address (check current gas prices on [Etherscan Gas Tracker](https://etherscan.io/gastracker) or the [OpenChainBench gas benchmark](https://openchainbench.com/benchmarks/gas-estimation); if gas prices are > 30gwei then we may need more) |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Use a direct gas-price tracker for the second reference.
The linked OpenChainBench page measures gas-oracle prediction accuracy and explicitly distinguishes that from selecting an oracle or reporting a current gas price. It may therefore mislead operators deciding how much ETH to fund. Replace it with a direct fee-estimate source or clarify exactly which metric should be used. (openchainbench.com)
🧰 Tools
🪛 LanguageTool
[grammar] ~188-~188: Ensure spelling is correct
Context: ...ks/gas-estimation); if gas prices are > 30gwei then we may need more) 2. Close the cur...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/deployment.md` at line 188, Update the funding instruction near the
Etherscan Gas Tracker reference by replacing the OpenChainBench link with a
direct current gas-price or fee-estimate source. Keep the existing guidance
about increasing the ETH amount when gas prices exceed 30 gwei.
1 participant
Summary
Line 188 of
docs/deployment.mdcurrently instructs deployers to check gas prices on ethgasstation.info, but that domain has been dead since around 2023:```
$ curl -sI https://ethgasstation.info/
HTTP/2 403
server: cloudflare
```
A deployer following this doc today lands on a Cloudflare-parked page and has no way to check gas prices as instructed.
Change
One line updated to point to two live alternatives: Etherscan Gas Tracker (widely used industry standard) and the OpenChainBench gas benchmark (independent multi-provider gas oracle accuracy comparison, CC BY 4.0 open data). No other changes.
Disclosure
I maintain OpenChainBench. Happy to swap the second reference for any other live gas oracle if that fits the doc style guide better, or drop it entirely and keep only the Etherscan Gas Tracker reference.
Summary by CodeRabbit