Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit bb3fec6

Browse filesBrowse files
fixes yogstation github guidelines outlined in contributing.md (yogstation13#16768)
* Update CONTRIBUTING.md * Update README.md * changes the yogstation github guidelines in the readme.md as a link so we only have to edit it in one place * Update README.md Co-authored-by: TheGamerdk <5618080+TheGamerdk@users.noreply.github.com> * Update CONTRIBUTING.md Co-authored-by: TheGamerdk <5618080+TheGamerdk@users.noreply.github.com>
1 parent 94b0acb commit bb3fec6
Copy full SHA for bb3fec6

2 files changed

+5-43Lines changed: 5 additions & 43 deletions

File tree

Expand file treeCollapse file tree
Open diff view settings
Filter options
Expand file treeCollapse file tree
Open diff view settings
Collapse file

‎.github/CONTRIBUTING.md‎

Copy file name to clipboardExpand all lines: .github/CONTRIBUTING.md
+4Lines changed: 4 additions & 0 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,8 @@ Maintainers can revert your changes if they feel they are not worth maintaining
5252

5353
**We allow users to create draft Pull Requests in order to have time to work on features and solicit feedback on those features**; however, there will be a limit of 2 draft Pull Requests per person, as we want people to finish their projects before moving onto others as soon as possible.
5454

55+
**In regards to Revert Pull Requests**, these should only be opened if there is a reason for the reversion ie. the feature is broken or is not as expected when it was merged. Otherwise please wait at least 48 hours before reversing a change.
56+
5557
### Maintainers
5658

5759
**As a maintainer, you are a representative of the development team, as such you should act with a somewhat professional manner when dealing with contributions, including constructively commenting on PRs.**
@@ -62,6 +64,8 @@ Maintainers can revert your changes if they feel they are not worth maintaining
6264
* A contributor has more than 2 draft Pull Requests open, in which the oldest draft Pull Request should be closed until the author closes another.
6365
* A contributor has opened a draft Pull Request that has no changes present in it after the initial 24 hours.
6466
* A contributor has not contributed to their draft Pull Request in a week’s time.
67+
* The Pull Request was opened with inadequate rationale or is lacking naming or following existing guidelines.
68+
6569

6670
**If a Maintainer/Director gets banned from the server/Discord, there will be an automatic review process triggered.** During this time access to GitHub merging and in-game ranks will be removed until the review is complete. After this review, roles and permissions may be returned depending on the result.
6771

Collapse file

‎README.md‎

Copy file name to clipboardExpand all lines: README.md
+1-43Lines changed: 1 addition & 43 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -22,50 +22,8 @@
2222

2323
[**Discord**](https://discord.gg/0keg6hQH05Ha8OfO)
2424

25-
<details><summary>Click here to view the Yogstation GitHub Guidelines</summary>
25+
[**Yogstation GitHub Guidelines**](https://github.com/yogstation13/Yogstation/blob/master/.github/CONTRIBUTING.md#yogstation-github-guidelines)
2626

27-
## Yogstation GitHub Guidelines
28-
29-
### General Rules
30-
31-
**Yogstation is an open source community-driven project that allows everybody to contribute their ideas towards making rounds on Space Station 13 as fun as possible by means of pull requests and issue reports to the main repository.**
32-
33-
**Although we allow conversation on GitHub in Pull Requests/Issues, we ask that all contributors and maintainers maintain a sense of decorum and respect the author and other people contributing to the discussion.** If someone is constantly contributing negative statements or is not constructive in their criticism they may be asked to stop or have their right to contribute removed in the future.
34-
35-
**The GitHub Terms and Conditions must be followed.** Failure to follow the terms and conditions of GitHub may result in your exclusion from contribution in the future.
36-
37-
**We do not require people to be signed up on the forums or Discord in order to contribute to the GitHub; however, there must be a way for maintainers to be able to contact you, either via Discord (preferably) or e-mail when concerns with identity are found and need to be resolved.** Being banned from the server does not automatically ban you from GitHub and vice versa; however, this is up to the discretion of the head developers. Your pull requests may all be placed on hold until you accept this request for communication.
38-
39-
### Contributors
40-
41-
**When you are creating a Pull Request/Issue Report**, please make sure you name the Pull Request/Issue something relevant to what is being changed, added, or removed. Do not attempt to mislead people about the content of the Pull Request/Issue. Make sure you fill out any applicable templates as clearly and concisely as possible, and link any relevant issues your Pull Request solves.
42-
43-
**We do not limit the amount of Pull Requests which can be submitted per day**; however, please do not submit more than is necessary. Additionally, please do not harass any member of the development team to expedite or merge your Pull Request. In certain circumstances, such as the case of a game breaking bug or issue, please reach out and inform them as soon as possible.
44-
45-
**We allow users to create draft Pull Requests in order to have time to work on features and solicit feedback on those features**; however, there will be a limit of 2 draft Pull Requests per person, as we want people to finish their projects before moving onto others as soon as possible.
46-
47-
### Maintainers
48-
49-
**As a maintainer, you are a representative of the development team, as such you should act with a somewhat professional manner when dealing with contributions, including constructively commenting on PRs.**
50-
51-
**All maintainers should follow set standards for handling Pull Requests.** These standards include waiting 24 hours before merging or closing a Pull Request, as well as merging only Pull Requests that fall under your area of expertise (i.e. an Art Maintainer should not merge code, and a Code Maintainer should not merge art.) Lastly, revert Pull Requests should only be merged by a Head Developer. The above limitations do not apply to round-breaking or repo-breaking changes; however, please notify any head developers if this occurs.
52-
53-
**All maintainers should encourage discourse and collaboration.** As such, maintainers should only close draft Pull Requests if:
54-
* A contributor has more than 2 draft Pull Requests open, in which the oldest draft Pull Request should be closed until the author closes another.
55-
* A contributor has opened a draft Pull Request that has no changes present in it after the initial 24 hours.
56-
* A contributor has not contributed to their draft Pull Request in a week’s time.
57-
58-
**If a Maintainer/Director gets banned from the server/Discord, there will be an automatic review process triggered.** During this time access to GitHub merging and in-game ranks will be removed until the review is complete. After this review, roles and permissions may be returned depending on the result.
59-
60-
### Contributor-Maintainer Disputes
61-
62-
**If you have any complaints about maintainers or contributors you can use the GitHub report function**; however, abuse of this feature will be addressed if needed.
63-
64-
**You may also raise an issue on discord by going to the #head-dev-complaints channel and creating a private thread**, then pinging head developers to make sure they are notified.
65-
66-
**These policies are enforced by the Head Developer(s) and are subject to change at their discretion, with or without notification to the general public.**
67-
68-
</details>
6927

7028

7129
## DOWNLOADING

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.