Skip to content

FIRE-823 | OpenSource | Add code of conduct#77

Merged
drorIvry merged 1 commit intomainfrom
feature/open-source-standards
Oct 12, 2025
Merged

FIRE-823 | OpenSource | Add code of conduct#77
drorIvry merged 1 commit intomainfrom
feature/open-source-standards

Conversation

@yuval-qf
Copy link
Collaborator

No description provided.

@yuval-qf yuval-qf requested a review from drorIvry October 12, 2025 15:03
@yuval-qf yuval-qf self-assigned this Oct 12, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 12, 2025

Summary by CodeRabbit

  • Documentation
    • Added a Contributor Covenant Code of Conduct outlining community standards, pledges, scope, enforcement responsibilities, and reporting process. Includes enforcement guidelines with graduated consequences and attribution to v2.0 resources. No functional or API changes. Improves clarity on acceptable behavior and moderation procedures for contributors and maintainers. Establishes expectations for respectful collaboration and conflict resolution.

Walkthrough

Added a new CODE_OF_CONDUCT.md document outlining community conduct, enforcement responsibilities and guidelines, scope, moderation procedures, and attribution to Contributor Covenant v2.0. No code or API changes.

Changes

Cohort / File(s) Change Summary
Community & Governance
CODE_OF_CONDUCT.md
Added Contributor Covenant-based Code of Conduct with pledges, standards, scope, enforcement responsibilities, guidelines, and attribution references.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I twitch my ears, approve with cheer,
A covenant clear for all to hear.
Be kind, be bright, let voices blend—
Carrots for care, and paws to mend.
In burrows of code, we hop as one—
With conduct set, the work is fun! 🥕🐇

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/open-source-standards

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

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.

Actionable comments posted: 0

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

63-64: Tighten formatting for lint compliance.

markdownlint is flagging bare URLs (MD034) and the document also has a double space before “harassment” on Line 109. Please convert the email/URLs to Markdown links (e.g., [admin@qualifire.ai](mailto:admin@qualifire.ai) and <https://…>), and trim the extra space to keep the doc tidy.

Also applies to: 109-110, 119-120, 127-128

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 90ef5db and 5c12481.

📒 Files selected for processing (1)
  • CODE_OF_CONDUCT.md (1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
CODE_OF_CONDUCT.md

63-63: Bare URL used

(MD034, no-bare-urls)


119-119: Bare URL used

(MD034, no-bare-urls)


127-127: Bare URL used

(MD034, no-bare-urls)


128-128: Bare URL used

(MD034, no-bare-urls)

⏰ 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). (2)
  • GitHub Check: codestyle
  • GitHub Check: rogue_sanity

@drorIvry drorIvry merged commit cfa4d44 into main Oct 12, 2025
8 checks passed
@drorIvry drorIvry deleted the feature/open-source-standards branch October 12, 2025 15:25
@drorIvry drorIvry mentioned this pull request Oct 28, 2025
21 tasks
@drorIvry drorIvry mentioned this pull request Nov 10, 2025
21 tasks
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