Skip to content

Conversation

@Titanium-Lung
Copy link
Contributor

NOT a change to the Constitution document.

Updated the CONTRIBUTING.md guide to be applicable to generic members and not just Maintainers, as well as changing the link to ComputerScienceHouse/Constitution, and removing mention of committee meetings.

@ikopke23
Copy link
Contributor

ikopke23 commented Feb 1, 2025

I like it just you forgot to remove your other PR so remove that

@Titanium-Lung Titanium-Lung force-pushed the update-contributing-guide branch from e46cf88 to 2aeae32 Compare February 1, 2025 21:16
@Titanium-Lung
Copy link
Contributor Author

Fixed it. It only has the commits to the CONTRIBUTING.md guide now

CONTRIBUTING.md Outdated
- `needs-discussion`: for issues that are ready for discussion at committee meetings.
- `discussion-issue`: for issues that contain longer-term discussions not ready for discussion at a committee meeting.
- `needs-discussion`: for issues that should be discussed with Maintainers.
- `discussion-issue`: for issues that contain longer-term discussions.
Copy link
Contributor

Choose a reason for hiding this comment

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

Containing a longer-term discussion is imo terrible wording and should be 'needs some long-term discussions.'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I changed it

Copy link
Contributor

@ikopke23 ikopke23 left a comment

Choose a reason for hiding this comment

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

I see nothing wrong afaik

@pikachu0542 pikachu0542 merged commit 279d4c2 into ComputerScienceHouse:master Feb 3, 2025
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.

3 participants