From aa213a9e027e2bc0d2bb08c66635c04de25d8a24 Mon Sep 17 00:00:00 2001 From: Noah Hanford Date: Sun, 12 Apr 2026 21:45:53 -0400 Subject: [PATCH 1/5] initial draft tm tm --- CONTRIBUTING.md | 12 ++++++++++++ 1 file changed, 12 insertions(+) create mode 100644 CONTRIBUTING.md diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md new file mode 100644 index 0000000..c94ce90 --- /dev/null +++ b/CONTRIBUTING.md @@ -0,0 +1,12 @@ +# Contributing to CSH +Hello! We love contributions, members who contribute are the best! + +In general, checkout the repository's [README.md](README.md) for specific instructions on +how to setup and run a service locally so that you can make your super cool awesome changes! + +After making your changes, please test as much as possible becuase we don't want to have broken code, and it's easier for everyone if +you realize bugs before someone else goes to test it too + +In general, AI use is okay if it's limited, but as an org of computer nerds for computer nerds, we feel that most of our code shouhld be written +by us. +For example: don't vibe code a service or a major change. Ultimately code you submit is owned by you and you should understand it :) From cb585cad208aff01683f20b23aee7378af9715e0 Mon Sep 17 00:00:00 2001 From: Noah Hanford Date: Sun, 12 Apr 2026 21:52:34 -0400 Subject: [PATCH 2/5] typos --- CONTRIBUTING.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index c94ce90..e6d16bb 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -4,9 +4,9 @@ Hello! We love contributions, members who contribute are the best! In general, checkout the repository's [README.md](README.md) for specific instructions on how to setup and run a service locally so that you can make your super cool awesome changes! -After making your changes, please test as much as possible becuase we don't want to have broken code, and it's easier for everyone if +After making your changes, please test as much as possible because we don't want to have broken code, and it's easier for everyone if you realize bugs before someone else goes to test it too -In general, AI use is okay if it's limited, but as an org of computer nerds for computer nerds, we feel that most of our code shouhld be written +In general, AI use is okay if it's limited, but as an org of computer nerds for computer nerds, we feel that most of our code should be written by us. -For example: don't vibe code a service or a major change. Ultimately code you submit is owned by you and you should understand it :) +Basically: don't vibe code a service or a major change. Ultimately code you submit is owned by you and you should understand it :) From 1a88d0a55b77442cadccdb8812afe97fd7b2db34 Mon Sep 17 00:00:00 2001 From: Noah Hanford Date: Sun, 12 Apr 2026 21:52:53 -0400 Subject: [PATCH 3/5] hm --- CONTRIBUTING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index e6d16bb..302c5f9 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -9,4 +9,4 @@ you realize bugs before someone else goes to test it too In general, AI use is okay if it's limited, but as an org of computer nerds for computer nerds, we feel that most of our code should be written by us. -Basically: don't vibe code a service or a major change. Ultimately code you submit is owned by you and you should understand it :) +Basically, don't vibe code a service or a major change. Ultimately code you submit is owned by you and you should understand it :) From 97d6a2e5fae275a2b19cbb886b87ab577163925e Mon Sep 17 00:00:00 2001 From: Noah Hanford Date: Sun, 12 Apr 2026 21:53:13 -0400 Subject: [PATCH 4/5] Update CONTRIBUTING.md --- CONTRIBUTING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 302c5f9..e86fd67 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -9,4 +9,4 @@ you realize bugs before someone else goes to test it too In general, AI use is okay if it's limited, but as an org of computer nerds for computer nerds, we feel that most of our code should be written by us. -Basically, don't vibe code a service or a major change. Ultimately code you submit is owned by you and you should understand it :) +Basically, don't vibe code a service or a major change. Ultimately code you submit is owned by you and you should understand it and be able to explain or defend it :) From 02d6cc37c6c5d6d2b3dcb6b9c1fd8c0d5f1f1669 Mon Sep 17 00:00:00 2001 From: Tyler Allen Date: Thu, 16 Apr 2026 21:15:11 -0400 Subject: [PATCH 5/5] Revised by OpComm Meeting Attendees Thank you @shaeespring @BigSpaceships @pikachu0542 for the contributions in the meeting! This is equally made by all of them. --- CONTRIBUTING.md | 41 +++++++++++++++++++++++++++++++++-------- 1 file changed, 33 insertions(+), 8 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index e86fd67..ef03888 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -1,12 +1,37 @@ -# Contributing to CSH +#Contributing to CSH + Hello! We love contributions, members who contribute are the best! -In general, checkout the repository's [README.md](README.md) for specific instructions on -how to setup and run a service locally so that you can make your super cool awesome changes! +When you start working with any repository, your first step should always be reading that repository's [README.md](README.md) for specific instructions on how to setup and run a service locally so that you can make your super cool awesome changes! + +All contributions must follow the contributions policies and CSH [Code of Conduct](https://coc.csh.rit.edu). Please use commit messages that align with both policies. We also recommend (but do not mandate) the use of [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) for commit message syntax. + + +##Generative AI Policy + +Using AI tools to help write your PR is acceptable, but as the author, you are responsible for understanding every change. If you used AI tools in preparing your PR, you must disclose this in the description of your PR. Listing AI tooling as a co-author, co-signing commits using an AI tool, or using the `assisted-by`, `co-developed` or similar commit trailer is not allowed. + +Large AI-generated PRs and AI-generated commit messages are not allowed. + +Do not leave the first review of AI generated changes to the reviewers. Verify the changes (code review, testing, etc.) before submitting your PR. You are responsible for checking bugs, checking user experience, ensuring no side effects occur, that the code passes linting. Reviewers can and will ask questions about any part of your PR, and if you cannot explain why a change was made, the PR will be closed. + +When responding to review comments, you must do so without relying on AI tools. Reviewers want to engage directly with you, not with generated responses. If you do not engage directly with reviewers, the PR will be closed. + +We are Computer Science House and one of the main parts of CSH is learning. This policy was created because we want to encourage all learning across the board. Banning generative AI (Gen AI) outright would remove the ability for CSHers to learn about Gen AI, the tooling commonly used in the industry, and workflow practices with the assistance of Gen AI tools. However, being an organization of computer nerds for computer nerds, the majority of our codebases should be written by members of CSH, and all of it should be understood by the member(s) authoring the PR. Gen AI can't sign a CoC, so it can't contribute to House Services. + + +##Documentation + +All contributors are expected to document their software so that future contributors can easily understand what the code does and how to contribute to it. +If you make changes to any services, and those changes make existing documentation outdated, you are expected to update the relevant documentation as part of your changes. This includes setup of the environment for testing and production, e.g. Compose files, environment variables, database configuration, and more. + + +##Pull Requests + +All changes should be submitted in a pull request (PR) to the development branch (should it exist, otherwise the default branch) of the relevant repository. If you make a PR to the wrong branch, you will be asked to change it before it gets reviewed. +Additionally, all repositories have a default PR template that should be filled out when creating the PR. If you do not fill the template out, we will ask you to do so before it gets reviewed. + -After making your changes, please test as much as possible because we don't want to have broken code, and it's easier for everyone if -you realize bugs before someone else goes to test it too +##Questions -In general, AI use is okay if it's limited, but as an org of computer nerds for computer nerds, we feel that most of our code should be written -by us. -Basically, don't vibe code a service or a major change. Ultimately code you submit is owned by you and you should understand it and be able to explain or defend it :) +If this document or a README is too confusing, reach out! You can make an issue on the GitHub repository or ask in #opcomm on Slack. #opcomm will probably have faster response time than opening an issue. If you make an issue on GitHub, it never hurts to drop a link in #opcomm as well!