-
-
Notifications
You must be signed in to change notification settings - Fork 974
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
fix: update discord link across the repository #3941
base: master
Are you sure you want to change the base?
Conversation
WalkthroughThis pull request focuses on updating Discord server invitation links across multiple documentation files and configuration settings. The changes involve replacing the old Discord server invite link Changes
Possibly related PRs
Poem
✨ Finishing Touches
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
🧹 Nitpick comments (1)
CONTRIBUTING.md (1)
6-6
: Consider improving the writing style.The phrase "feel free to" could be replaced with a more professional alternative, such as:
- "You're welcome to ping @rrousselGit..."
- "Please ping @rrousselGit..."
🧰 Tools
🪛 LanguageTool
[style] ~6-~6: The phrase ‘feel free to’ is used quite frequently. Consider using a less frequent alternative to set your writing apart from others and make it sound more professional.
Context: ...https://discord.gg/hUUQkd9v) to chat. Feel free to ping @rrousselGit there for questions r...(FEEL_FREE_TO_STYLE_ME)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
CONTRIBUTING.md
(1 hunks)packages/riverpod_annotation/README.md
(1 hunks)packages/riverpod_generator/README.md
(2 hunks)packages/riverpod_lint/README.md
(1 hunks)website/docusaurus.config.js
(1 hunks)website/i18n/ko/docusaurus-theme-classic/footer.json
(1 hunks)website/i18n/zh-Hans/docusaurus-theme-classic/footer.json
(1 hunks)
✅ Files skipped from review due to trivial changes (3)
- website/i18n/zh-Hans/docusaurus-theme-classic/footer.json
- packages/riverpod_annotation/README.md
- packages/riverpod_lint/README.md
🧰 Additional context used
🪛 LanguageTool
CONTRIBUTING.md
[style] ~6-~6: The phrase ‘feel free to’ is used quite frequently. Consider using a less frequent alternative to set your writing apart from others and make it sound more professional.
Context: ...https://discord.gg/hUUQkd9v) to chat. Feel free to ping @rrousselGit there for questions r...
(FEEL_FREE_TO_STYLE_ME)
⏰ Context from checks skipped due to timeout of 90000ms (19)
- GitHub Check: Redirect rules - river-pod
- GitHub Check: Header rules - river-pod
- GitHub Check: Pages changed - river-pod
- GitHub Check: build (master, packages/riverpod_generator)
- GitHub Check: build (master, packages/hooks_riverpod/example)
- GitHub Check: build (master, packages/hooks_riverpod)
- GitHub Check: build (master, packages/flutter_riverpod/example)
- GitHub Check: build (master, packages/flutter_riverpod)
- GitHub Check: build (master, packages/riverpod)
- GitHub Check: build (master, examples/todos)
- GitHub Check: build (master, examples/stackoverflow)
- GitHub Check: build (master, examples/pub)
- GitHub Check: build (master, examples/marvel)
- GitHub Check: build (master, examples/counter)
- GitHub Check: riverpod_lint (stable, packages/riverpod_lint_flutter_test)
- GitHub Check: riverpod_lint (stable, packages/riverpod_analyzer_utils_tests)
- GitHub Check: riverpod_lint (master, packages/riverpod_lint_flutter_test)
- GitHub Check: riverpod_lint (master, packages/riverpod_analyzer_utils_tests)
- GitHub Check: check_generation
🔇 Additional comments (5)
website/docusaurus.config.js (1)
79-79
: LGTM! Discord link updated correctly.The Discord link has been updated consistently in the footer configuration.
website/i18n/ko/docusaurus-theme-classic/footer.json (1)
24-24
: LGTM! Discord link description updated correctly in Korean translation.The Discord link has been updated consistently in the Korean footer translation.
packages/riverpod_generator/README.md (2)
6-6
: LGTM! Discord badge link updated correctly.The Discord badge link has been updated consistently in the README.
89-92
: Verify if TOC changes are intended for this PR.The changes to the table of contents (adding
riverpod_generator
andGlobal configuration
sections) appear to be unrelated to the Discord link update PR objective.CONTRIBUTING.md (1)
6-6
: LGTM! Discord link updated correctly.The Discord link has been updated consistently in the contributing guide.
🧰 Tools
🪛 LanguageTool
[style] ~6-~6: The phrase ‘feel free to’ is used quite frequently. Consider using a less frequent alternative to set your writing apart from others and make it sound more professional.
Context: ...https://discord.gg/hUUQkd9v) to chat. Feel free to ping @rrousselGit there for questions r...(FEEL_FREE_TO_STYLE_ME)
closes #3942 |
Description
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]
).I have updated the
CHANGELOG.md
of the relevant packages.Changelog files must be edited under the form:
If this contains new features or behavior changes,
I have updated the documentation to match those changes.
Summary by CodeRabbit
Documentation
Chores
Bbumvej
tohUUQkd9v