-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
GH-55 Add README.md #55
base: master
Are you sure you want to change the base?
Conversation
@coderabbitai review |
@coderabbitai full review |
Warning Rate limit exceeded@Rollczi has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 18 minutes and 19 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. 📝 Walkthrough📝 Walkthrough📝 Walkthrough📝 WalkthroughWalkthroughThe changes in this pull request involve the introduction of a new README.md file for the Multification library, which is tailored for creating customizable notifications and messages in Minecraft plugins. The README includes a banner image, links to Patreon, the official website, and Discord, and outlines the library's capabilities, such as sending various types of notifications (chat messages, titles, action bars, etc.) and supporting multiple player targets and console messaging. Additionally, the README provides a "Getting Started" section with integration instructions, including necessary entries for the In the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
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
CodeRabbit Configuration File (
|
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
🧹 Outside diff range and nitpick comments (3)
README.md (3)
3-3
: Add alt text to the banner image.Hey there! It'd be great if you could add some alt text to the banner image. This helps make your README more accessible to everyone.
🧰 Tools
🪛 Markdownlint
3-3: null
Images should have alternate text (alt text)(MD045, no-alt-text)
99-99
: Let's polish this sentence.This sentence could use a bit of tweaking. How about:
"Setting up configuration is easy in both CDN and Okaeri configs. To add messages to the configuration, create a variable in the config with the classNotice
orBukkitNotice
. You can also use a builder. After plugin deployment, you can find messages in the configuration file."🧰 Tools
🪛 LanguageTool
[uncategorized] ~99-~99: You might be missing the article “a” here.
Context: ...d messages to the configuration, create variable in config with classNotice
or `Bukkt...(AI_EN_LECTOR_MISSING_DETERMINER_A)
[uncategorized] ~99-~99: You might be missing the article “the” here.
Context: ...uration, create variable in config with classNotice
orBukktiNotice
. You can als...(AI_EN_LECTOR_MISSING_DETERMINER_THE)
[uncategorized] ~99-~99: Possible missing comma found.
Context: ... You can also use builder. After plugin deploy you can find messages in configuration ...(AI_HYDRA_LEO_MISSING_COMMA)
101-101
: Quick heading fix.Let's remove the period at the end of this heading to keep it consistent with Markdown style.
🧰 Tools
🪛 Markdownlint
101-101: Punctuation: '.'
Trailing punctuation in heading(MD026, no-trailing-punctuation)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
⛔ Files ignored due to path filters (1)
assets/readme-banner.png
is excluded by!**/*.png
📒 Files selected for processing (3)
- README.md (1 hunks)
- buildSrc/src/main/kotlin/Versions.kt (1 hunks)
- examples/bukkit/src/main/java/com/eternalcode/example/bukkit/config/MessagesConfig.java (1 hunks)
🧰 Additional context used
🪛 LanguageTool
README.md
[uncategorized] ~99-~99: You might be missing the article “a” here.
Context: ...d messages to the configuration, create variable in config with classNotice
or `Bukkt...(AI_EN_LECTOR_MISSING_DETERMINER_A)
[uncategorized] ~99-~99: You might be missing the article “the” here.
Context: ...uration, create variable in config with classNotice
orBukktiNotice
. You can als...(AI_EN_LECTOR_MISSING_DETERMINER_THE)
[uncategorized] ~99-~99: Possible missing comma found.
Context: ... You can also use builder. After plugin deploy you can find messages in configuration ...(AI_HYDRA_LEO_MISSING_COMMA)
🪛 Markdownlint
README.md
101-101: Punctuation: '.'
Trailing punctuation in heading(MD026, no-trailing-punctuation)
3-3: null
Images should have alternate text (alt text)(MD045, no-alt-text)
🔇 Additional comments (5)
buildSrc/src/main/kotlin/Versions.kt (1)
5-5
: Hey there! Just checking on this version change.I noticed you changed the ADVENTURE_API version. Is this intentional? It might be good to double-check if everything still works smoothly with this older version.
Let's make sure this change doesn't cause any hiccups:
✅ Verification successful
ADVENTURE_API Version Update Verified ✅
I've confirmed that the ADVENTURE_API version "4.17.0" is consistently used throughout the codebase. Everything looks good!
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Check if the new version is used consistently rg "4.17.0" --type kotlinLength of output: 101
examples/bukkit/src/main/java/com/eternalcode/example/bukkit/config/MessagesConfig.java (1)
25-25
: Nice color update!The subtitle now uses a cool gradient instead of the previous tag. It'll look prettier for players!
README.md (3)
15-30
: Great intro! Consider a small tweak.The introduction looks good! It might be worth tweaking line 17 as suggested earlier to make it even clearer.
32-39
: Setup looks good, but could use more details.The setup instructions are clear, but it might be helpful to add more info about CDN/Okaeri configuration as suggested earlier.
41-97
: Code examples look great!The code examples are really helpful. Don't forget to remove 'private' from line 83 as suggested earlier.
Co-authored-by: Jakubk15 <[email protected]>
Co-authored-by: Jakubk15 <[email protected]>
Co-authored-by: Jakubk15 <[email protected]>
Co-authored-by: Jakubk15 <[email protected]>
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
🧹 Outside diff range and nitpick comments (2)
README.md (2)
3-3
: Add alt text to the banner imageAdding alt text will make the README more accessible for screen readers.
-![](/assets/readme-banner.png) +![Multification Banner](/assets/readme-banner.png)🧰 Tools
🪛 Markdownlint
3-3: null
Images should have alternate text (alt text)(MD045, no-alt-text)
37-45
: Consider adding version compatibility informationIt would be helpful to specify which Minecraft versions are supported by the library.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
⛔ Files ignored due to path filters (1)
assets/readme-banner.png
is excluded by!**/*.png
📒 Files selected for processing (2)
README.md
(1 hunks)examples/bukkit/src/main/java/com/eternalcode/example/bukkit/config/MessagesConfig.java
(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- examples/bukkit/src/main/java/com/eternalcode/example/bukkit/config/MessagesConfig.java
🧰 Additional context used
🪛 LanguageTool
README.md
[uncategorized] ~142-~142: Possible missing comma found.
Context: ...ile, use the following code in the init method such as onEnable
: ```java MessagesCo...
(AI_HYDRA_LEO_MISSING_COMMA)
🪛 Markdownlint
README.md
118-118: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
124-124: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
132-132: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
140-140: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
159-159: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
173-173: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
182-182: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
3-3: null
Images should have alternate text (alt text)
(MD045, no-alt-text)
🔇 Additional comments (2)
README.md (2)
20-32
: Great job with the feature list!
The emoji usage makes the features stand out nicely, and the descriptions are clear and concise.
108-192
: Configuration documentation looks complete
The setup instructions and examples for both CDN and Okaeri are clear and well-documented. This addresses the previous review comment about better configuration explanation.
🧰 Tools
🪛 LanguageTool
[uncategorized] ~142-~142: Possible missing comma found.
Context: ...ile, use the following code in the init method such as onEnable
: ```java MessagesCo...
(AI_HYDRA_LEO_MISSING_COMMA)
🪛 Markdownlint
118-118: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
124-124: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
132-132: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
140-140: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
159-159: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
173-173: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
182-182: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
No description provided.