Skip to content

Comments

docs: communicate professionally and avoid flattery.#89

Merged
markholdex merged 3 commits intomainfrom
zolotokrylin-patch-1
Feb 16, 2026
Merged

docs: communicate professionally and avoid flattery.#89
markholdex merged 3 commits intomainfrom
zolotokrylin-patch-1

Conversation

@zolotokrylin
Copy link
Member

@zolotokrylin zolotokrylin commented Feb 16, 2026

Updated the Code of Conduct to enhance communication guidelines and clarify core values.

Summary by CodeRabbit

  • Documentation
    • Updated Code of Conduct with enhanced core values and expanded communication guidelines, providing clearer expectations for community interactions.

Updated the Code of Conduct to enhance communication guidelines and clarify core values.

Signed-off-by: Vadim <vadim@holdex.io>
@zolotokrylin zolotokrylin self-assigned this Feb 16, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 16, 2026

Walkthrough

Updates docs/CODE_OF_CONDUCT.md by replacing concise, numeric guidelines with expanded, bolded core values and detailed communication expectations. Adds emphasis on clarified directives while removing terse phrasing.

Changes

Cohort / File(s) Summary
Documentation Update
docs/CODE_OF_CONDUCT.md
Reformatted Core Values with bolded titles, expanded explanations, and new communication-specific guidance. Replaced numeric mentions and one-liner directives with detailed expectations for notifications and daily updates.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Suggested reviewers

  • matthewvogas
  • angelicawill
  • dante4rt
  • markholdex
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly relates to the main change: updating Code of Conduct guidelines to emphasize professional communication and discourage flattery, which aligns with the expanded communication rules and core values in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch zolotokrylin-patch-1

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@zolotokrylin zolotokrylin marked this pull request as ready for review February 16, 2026 07:08
@holdex
Copy link

holdex bot commented Feb 16, 2026

Time Submission Status

Member Status Time Action Last Update
zolotokrylin ✅ Submitted 1h Update time Feb 16, 2026, 1:53 PM
markholdex ✅ Submitted 5min Update time Feb 16, 2026, 7:17 AM

You can submit time with the command. Example:

@holdex pr submit-time 15m

See available commands to help comply with our Guidelines.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
docs/CODE_OF_CONDUCT.md (1)

36-36: Consider a tighter, more specific phrasing.

LanguageTool suggests a more descriptive phrasing for the “Keep the Focus” question. If you want, refine the sentence to make the intent sharper.

@markholdex markholdex merged commit e95e932 into main Feb 16, 2026
5 checks passed
@markholdex markholdex deleted the zolotokrylin-patch-1 branch February 16, 2026 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants