Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nerf crates to carry the default entity limit of 30 entities #24

Merged
merged 1 commit into from
Jul 12, 2023
Merged

Nerf crates to carry the default entity limit of 30 entities #24

merged 1 commit into from
Jul 12, 2023

Conversation

Partmedia
Copy link
Contributor

Per #bulletin-board "lower capacity of crates", cherry-picked from mining-station-14

@dvir001
Copy link
Contributor

dvir001 commented Jul 11, 2023

This change is too big, IMO need to be more like 100-200 but not 30

@Partmedia
Copy link
Contributor Author

Happy to change this to whatever we think the best number is.

@Cheackraze Cheackraze self-requested a review July 12, 2023 06:05
Copy link
Member

@Cheackraze Cheackraze left a comment

Choose a reason for hiding this comment

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

thats still 30 stacks, im willing to try it, worst thing that could happen is maybe some pre-fills like bar supply crate might spawn funny but it would be an easy revert regardless

@Cheackraze Cheackraze merged commit b059794 into new-frontiers-14:master Jul 12, 2023
7 of 13 checks passed
@Partmedia Partmedia deleted the f/crates branch July 18, 2023 19:29
diatomic-ge pushed a commit to diatomic-ge/frontier-station-14 that referenced this pull request Jan 6, 2024
Implements the NCWL softsuit, implements new lobby music. Fixes .yml …
dvir001 pushed a commit to dvir001/frontier-station-14 that referenced this pull request Aug 24, 2024
…r grid check. (new-frontiers-14#24)

* Make APCs explode after a random delay during a power grid check.

* Change buzz message according to review

* Make message hidden in chat

* Increase delay between buzzes

* Named parameters

---------

Co-authored-by: TsjipTsjip <[email protected]>
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