-
Notifications
You must be signed in to change notification settings - Fork 48
feat: court llms.txt #2067
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
feat: court llms.txt #2067
Conversation
✅ Deploy Preview for kleros-v2-testnet-devtools ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
❌ Deploy Preview for kleros-v2-university failed. Why did it fail? →
|
Caution Review failedThe pull request is closed. WalkthroughA new documentation file, Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
✅ Deploy Preview for kleros-v2-testnet ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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: 0
🧹 Nitpick comments (5)
web/src/public/llms.txt (2)
1-3
: Prefer.md
extension for Markdown assetsThe file uses Markdown syntax but is saved as
.txt
. Renaming it tollms.md
improves automatic rendering, syntax highlighting, and GitHub preview.
5-9
: Reduce repetitive lead-in wording in list itemsFive successive bullets start with “Kleros …”, triggering the style-checker and adding noise. Dropping the prefix or shortening it makes the list crisper:
- [Kleros Dispute Dashboard]... - [Kleros Dispute Cases]... - [Kleros Decentralized Courts]... - [Kleros Jurors Leaderboard]... - [Get PNK Token]... + [Dispute Dashboard]... + [Cases]... + [Decentralized Courts]... + [Juror Leaderboard]... + [Get PNK]...web/src/public/llms-full.txt (3)
1-1
: Rename to.md
for proper Markdown handlingSame rationale as in
llms.txt
: a.md
extension will let tooling and browsers render the document instead of serving raw text.
19-24
: Duplicate “¿Debería la empresa pagar la compensación exigida por el usuario?” entryLines 19 and 23 reference the same case ID 83 and 81 respectively, but the titles are identical, which looks accidental and may confuse readers. Keep only distinct cases or clarify the difference.
31-34
: Minor grammar adjustmentReplace “times where” with “times the juror” (or “times when”) and tweak wording for clarity.
-A score from 0 to 100 reflecting coherent voting, smoothed to prevent jurors with low vote counts from ranking too high. - -This is the ratio of coherent votes made by a juror: the number in the left is the number of times where the juror voted coherently and the number in the right is the total number of times the juror voted. Hover to see the percentage of coherent votes. +A score from 0 to 100 reflects coherent voting and is smoothed so jurors with few votes do not rank disproportionately high. + +The ratio shows coherent votes made by a juror: the number on the left is how many times the juror voted coherently, and the number on the right is the total number of votes cast. Hover to see the percentage.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
web/src/public/llms-full.txt
(1 hunks)web/src/public/llms.txt
(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: in the kleros court configuration, it's acceptable for the general court (id: 1) to have itself as i...
Learnt from: jaybuidl
PR: kleros/kleros-v2#1746
File: contracts/config/courts.v2.mainnet-neo.json:3-17
Timestamp: 2024-11-19T16:09:52.022Z
Learning: In the Kleros court configuration, it's acceptable for the General Court (id: 1) to have itself as its parent (parent: 1) when there is no court at index 0.
Applied to files:
web/src/public/llms-full.txt
🪛 LanguageTool
web/src/public/llms.txt
[style] ~7-~7: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...entralized dispute resolution cases. - [Kleros Decentralized Courts](https://v2.kleros...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[style] ~8-~8: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ate as jurors, and view court cases. - [Kleros Jurors Leaderboard](https://v2.kleros.b...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
web/src/public/llms-full.txt
[style] ~33-~33: It might be better to use ‘times’ with the time-relative pronoun ‘when’. (Alternatively, use ‘in/on which’.)
Context: ...the number in the left is the number of times where the juror voted coherently and the numb...
(WHEN_WHERE)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (14)
- GitHub Check: Redirect rules - kleros-v2-testnet
- GitHub Check: Redirect rules - kleros-v2-testnet
- GitHub Check: Header rules - kleros-v2-testnet
- GitHub Check: Header rules - kleros-v2-testnet
- GitHub Check: Pages changed - kleros-v2-testnet
- GitHub Check: Pages changed - kleros-v2-testnet
- GitHub Check: Redirect rules - kleros-v2-university
- GitHub Check: Header rules - kleros-v2-university
- GitHub Check: Redirect rules - kleros-v2-testnet-devtools
- GitHub Check: Pages changed - kleros-v2-university
- GitHub Check: Header rules - kleros-v2-testnet-devtools
- GitHub Check: Pages changed - kleros-v2-testnet-devtools
- GitHub Check: contracts-testing
- GitHub Check: Analyze (javascript)
✅ Deploy Preview for kleros-v2-neo ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
2dde0d8
to
752b64a
Compare
|
PR-Codex overview
This PR focuses on updating the
netlify.toml
configuration for headers and adding content to thellms.txt
file, which describes the Kleros platform and its features.Detailed summary
netlify.toml
to setX-Robots-Tag
for all routes.llms.txt
with descriptions of Kleros features including:Summary by CodeRabbit