-
Notifications
You must be signed in to change notification settings - Fork 23
Explorer and Docs by WHTech #44
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
base: main
Are you sure you want to change the base?
Conversation
WalkthroughThe README.md was updated to replace all AXONE-related documentation and block explorer references with OKP4-specific resources. This includes new tables for documentation and explorers, Indonesian column headers, updated URLs, contributor information, and status indicators. No code or exported/public entities were changed; only the documentation content was modified. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. ✨ 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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
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.
Actionable comments posted: 1
🧹 Nitpick comments (4)
README.md (4)
43-44: Unify column language with the rest of the READMEAll earlier headings are in English; the new table headers switch to Indonesian (“Judul”, “Sumber”). For consistency across the doc, consider translating them to English (e.g., “Title”, “Source”).
47-60: Consolidate duplicated “Nodes & Validator Guide” rowsSeven rows share the exact same title but point to different guides. A more reader-friendly approach is to collapse them into one “Nodes & Validator Guides” row and list the contributing sites in the “Source” column (comma-separated), or give each a distinctive title (e.g., “Nodes & Validator Guide – Nodes Guru”).
52-52: Typo: “Sanpshot” → “Snapshot”-| OKP4 Snapshot Guide | https://github.com/Kolot86/My-guides/blob/main/OKP4_Sanpshot_Guide/OKP4.md | ✅ | [Kolot](https://github.com/Kolot86) | +| OKP4 Snapshot Guide | https://github.com/Kolot86/My-guides/blob/main/OKP4_Snapshot_Guide/OKP4.md | ✅ | [Kolot](https://github.com/Kolot86) |
62-63: Remove superfluous blank lineThere are two consecutive blank lines after the documentation table (md-lint MD012). Delete one to keep markdownlint happy.
- - +
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md(1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
README.md
⚙️ CodeRabbit Configuration File
This is an "awesome" repository — a curated list of valuable resources. Review the Markdown content with a focus on editorial quality and curation intent. For entries in tables, adapt to the format: if a description column is present, it should be meaningful and clear. Ensure all content aligns with the spirit of a well-maintained, community-driven list.
Files:
README.md
🧠 Learnings (2)
📓 Common learnings
Learnt from: ccamel
PR: axone-protocol/awesome#4
File: README.md:47-50
Timestamp: 2025-07-12T12:39:32.704Z
Learning: In the axone-protocol/awesome repository, empty tables (like the "Block Explorers" section) are intentionally kept as placeholders for future community contributions rather than being removed or commented out.
README.md (1)
Learnt from: ccamel
PR: #4
File: README.md:47-50
Timestamp: 2025-07-12T12:39:32.704Z
Learning: In the axone-protocol/awesome repository, empty tables (like the "Block Explorers" section) are intentionally kept as placeholders for future community contributions rather than being removed or commented out.
🪛 markdownlint-cli2 (0.17.2)
README.md
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
52-52: Bare URL used
(MD034, no-bare-urls)
53-53: Bare URL used
(MD034, no-bare-urls)
54-54: Bare URL used
(MD034, no-bare-urls)
55-55: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
62-62: Multiple consecutive blank lines
Expected: 1; Actual: 2
(MD012, no-multiple-blanks)
67-67: Bare URL used
(MD034, no-bare-urls)
68-68: Bare URL used
(MD034, no-bare-urls)
69-69: Bare URL used
(MD034, no-bare-urls)
70-70: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
🔇 Additional comments (1)
README.md (1)
65-77: Verify explorer URLs and deduplicate WHTech row
- Two consecutive rows list WHTech with different explorer domains. If both are required, clarify the difference in the “Explorer” column (e.g., “Ping Pub (whtech)” vs “Custom”). Otherwise, remove the duplicate.
https://ping.pub/okp4%20testnet?testnetcontains%20; the space may break the link. Please test and correct if needed.-| [WHTech](https://whtech.xyz) | Ping Pub | https://explorer.whtech.xyz | ✅ | -| [WHTech](https://whtech.xyz) | WHTech | https://explorerku.whtech.xyz | ✅ | +| [WHTech](https://whtech.xyz) | Ping Pub | https://explorer.whtech.xyz | ✅ | +| [WHTech](https://whtech.xyz) | Custom | https://explorerku.whtech.xyz | ✅ |
| | Judul | URL | Status | Sumber | | ||
| |----------------------------------------|----------------------------------------------------------------------------------------|--------|------------------------------------------------------------------------------------------| | ||
| | Whitepaper | https://docs.okp4.network/whitepaper/abstract | ✅ | [OKP4](https://github.com/okp4) | | ||
| | Nodes & Validator Guide | https://docs.okp4.network/nodes/introduction | ❌ | [OKP4](https://github.com/okp4) | | ||
| | Nodes & Validator Guide | https://nodes.guru/okp4/setup-guide/en | ✅ | [Nodes Guru](https://nodes.guru/) | | ||
| | Nodes & Validator Guide | https://nodeist.net/t/Okp4/Setup/ | ✅ | [Nodeist](https://nodeist.net/) | | ||
| | Nodes & Validator Guide | https://nodeservice.nodersteam.com/title/testnets/okp4 | ❌ | [[NODERS]TEAM](https://noders-stake.com/) | | ||
| | Nodes & Validator Guide | https://aviaone.com/blockchains-service/okp4-testnet-okp4-nemeton-1.html | ✅ | [AVIAONE](https://aviaone.com) | | ||
| | Dev Guidelines 👩💻 | https://work.okp4.com/dev/dev-home | ✅ | [OKP4](https://github.com/okp4) | | ||
| | OKP4 Snapshot Guide | https://github.com/Kolot86/My-guides/blob/main/OKP4_Sanpshot_Guide/OKP4.md | ✅ | [Kolot](https://github.com/Kolot86) | | ||
| | Basic Tools & Validator Guide | https://github.com/Cumulo-pro/OKP4#readme | ✅ | [Cumulo](https://github.com/Cumulo-pro) | | ||
| | FAQ OKP4 Validator | https://medium.com/cumulo-pro/faq-okp4-validator-19e81661b101 | ✅ | [Cumulo](https://github.com/Cumulo-pro) | | ||
| | OKP4 Community of Validators Resources | https://medium.com/cumulo-pro/okp4-community-of-validators-resources-f3e5b2b4d2f7 | ✅ | [Cumulo](https://github.com/Cumulo-pro) | | ||
| | TMKMS Guide | https://craving-for-knowledge.gitbook.io/craving_for_knowledge/testnets/okp4/tmkms | ✅ | [Craving_for_Knowledge](https://craving-for-knowledge.gitbook.io/craving_for_knowledge/) | | ||
| | Nodes & Validator Guide | https://github.com/cryptobtcbuyer/Testnet_guides/blob/main/OKP4/installation.md | ✅ | [cryptobtcbuyer](https://github.com/cryptobtcbuyer) | | ||
| | Monitoring & Alerting for Validators | https://github.com/cryptobtcbuyer/Testnet_guides/blob/main/OKP4/monitoring.md | ✅ | [cryptobtcbuyer](https://github.com/cryptobtcbuyer) | | ||
| | Nodes & Validator Guide | https://nodestake.top/okp4 | ❌ | [NodeStake](https://nodestake.top/okp4) | | ||
| | WHTech Docs | https://docs.whtech.xyz | ❌ | [WHTech](https://github.com/WHTechno) | |
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.
🛠️ Refactor suggestion
Consider scoping OKP4 resources in an Axone-centric list
The section suddenly pivots from Axone to OKP4 documentation. Because the repo brands itself as “Awesome AXONE,” readers may be confused about the relevance of OKP4 material. Either:
- Create a dedicated OKP4 subsection (clearly labelled as “Related ecosystems”) – or
- Move these entries to a separate awesome list and keep this file Axone-only.
This keeps the curation crisp and mission-focused.
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
52-52: Bare URL used
(MD034, no-bare-urls)
53-53: Bare URL used
(MD034, no-bare-urls)
54-54: Bare URL used
(MD034, no-bare-urls)
55-55: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
🤖 Prompt for AI Agents
In README.md between lines 43 and 60, the list mixes Axone and OKP4 resources,
which may confuse readers. To fix this, create a clearly labeled subsection for
OKP4 resources, such as "Related ecosystems," separating them from Axone
content, or move all OKP4 entries to a separate awesome list to maintain focus
on Axone in this file.
ccamel
left a comment
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.
Hmm, this change isn't correct. It looks like you're importing updates from the okp4/awesome repository. You should rebase the branch to include only the actual changes relative to the main branch of axone-protocol/awesome.
Let me know if you need a hand.
|
yes sir, i can't know step by step PR from archieve, i just add link my contribution |
Summary by CodeRabbit