Skip to content

Contributing Guidelines Draft#2

Draft
BigSpaceships wants to merge 5 commits intomainfrom
contributing.md
Draft

Contributing Guidelines Draft#2
BigSpaceships wants to merge 5 commits intomainfrom
contributing.md

Conversation

@BigSpaceships
Copy link
Copy Markdown

@BigSpaceships BigSpaceships commented Apr 13, 2026

What

Adds a contributing.md with initial guidelines

Checklist

  • Tested all changes locally

costowell
costowell previously approved these changes Apr 13, 2026
Copy link
Copy Markdown

@tallen42 tallen42 left a comment

Choose a reason for hiding this comment

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

This should go in front of house for a discussion before it is merged in.

@BigSpaceships BigSpaceships changed the title initial draft tm tm Contributing Guidelines Draft Apr 15, 2026
@mbillow
Copy link
Copy Markdown
Member

mbillow commented Apr 15, 2026

In general, I think this might be a little bit too far. Personally, I really like the approach the Kubernetes project (and various other LF/CNCF projects) have adopted: https://groups.google.com/a/kubernetes.io/g/dev/c/7Y9016gdFZw/m/nUUmxFPzAQAJ

tl;dr: If it is AI, say so. Make sure it works and make sure you understand it. Be ready for questions or be ready for rejection.

Learning to wield generative AI effectively is just as important as learning the concepts it is generating. I agree engineers should walk (code) before they run (vibe), in order to ensure they are designing the best systems. But this policy feels a little too black and white to support both ends of that spectrum.

@pikachu0542
Copy link
Copy Markdown
Member

pikachu0542 commented Apr 17, 2026

We should add stuff about the expectation to document your software and update any documentation that you make outdated

Comment thread CONTRIBUTING.md Outdated
Thank you @shaeespring @BigSpaceships @pikachu0542 for the contributions in the meeting! This is equally made by all of them.
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.

6 participants