Skip to content

chore: add Code of Conduct - #1050

Merged
mbevc1 merged 4 commits into
mainfrom
mbevc1-patch-1
Jul 28, 2026
Merged

chore: add Code of Conduct#1050
mbevc1 merged 4 commits into
mainfrom
mbevc1-patch-1

Conversation

@mbevc1

@mbevc1 mbevc1 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Add Code of Conduct

Checklist

@mbevc1 mbevc1 added the documentation Improvements or additions to documentation label Jul 28, 2026
@claude

claude Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Claude finished @mbevc1's task in 54s —— View job


Code review: Code of Conduct

  • Read the changed file (CODE_OF_CONDUCT.md)
  • Assess static-analysis needs
  • Review content & consistency
  • Post feedback

This PR adds a standard Contributor Covenant Code of Conduct. It's a documentation-only change (one new Markdown file), so the Go static-analysis phase (go build / go vet / go mod tidy) doesn't apply — nothing in the build is affected. 👍

Findings

1. Inconsistent line-wrapping (minor)CODE_OF_CONDUCT.md:106
The rest of the document is hard-wrapped at ~80 columns, but the "Permanent Ban" paragraph is a single long unwrapped line. I've left an inline suggestion to reflow it for consistency and cleaner future diffs.

2. Enforcement contact (worth confirming, non-blocking)CODE_OF_CONDUCT.md:61
Conduct reports are routed to security@kosli.com, the same address used in SECURITY.md for vulnerability disclosure. That's a valid choice, but worth double-checking it's the intended inbox for conduct reports rather than a dedicated alias — the two report types often go to different people.

3. Version vs. wording (informational)
The Attribution section cites Contributor Covenant v2.0, but the "Permanent Ban" wording (...for which a temporary ban is not deemed sufficient OR demonstrating a pattern...) matches v2.1. Not a problem for a customized CoC, just flagging that the cited version and the text have drifted slightly if you want them to line up.

Nice work

  • Clean, complete adoption of a well-recognized standard.
  • Spelling ("behaviour", "apologizing") is applied consistently throughout.
  • Providing a real enforcement contact rather than leaving the template placeholder is the right call.

Overall this is good to merge once the wrapping nit is addressed (or intentionally skipped).
• branch mbevc1-patch-1

Comment thread CODE_OF_CONDUCT.md Outdated
@mbevc1
mbevc1 enabled auto-merge (squash) July 28, 2026 14:51
Comment thread CODE_OF_CONDUCT.md Outdated
mbevc1 and others added 2 commits July 28, 2026 15:51
Co-authored-by: Marko Bevc <marko.bevc@gmail.com>
Co-authored-by: AlexKantor87 <alex@kosli.com>
Comment thread CODE_OF_CONDUCT.md Outdated
@kosli-dev kosli-dev deleted a comment from claude Bot Jul 28, 2026
@kosli-dev kosli-dev deleted a comment from mbevc1 Jul 28, 2026
Co-authored-by: AlexKantor87 <alex@kosli.com>
Comment thread CODE_OF_CONDUCT.md
@mbevc1
mbevc1 merged commit 7a13dcc into main Jul 28, 2026
20 checks passed
@mbevc1
mbevc1 deleted the mbevc1-patch-1 branch July 28, 2026 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants