Skip to content
This repository has been archived by the owner on Aug 21, 2023. It is now read-only.

Moves experimental mining disks to blood drunk miner #850

Closed

Conversation

Gboster-0
Copy link

@Gboster-0 Gboster-0 commented Dec 8, 2022

About The Pull Request

Moves experimental PKA disks from necropolis chests to the blood drunk miner
The miner will drop 1 disk, meaning the other 3 are unobtainable for the shift

Why It's Good For The Game

Literally nobody likes them, i have never seen a single person be happy from having them in the necropolis chest and frankly they should just not be there. Instead moved them to the blood-drunk miner so they arent clogging up random loot in lavaland chests

Changelog

🆑

tweak: Experimental mining PKA disks moved to blood drunk

/:cl:

@Gboster-0 Gboster-0 requested a review from nednaZ December 15, 2022 03:55
@nednaZ nednaZ added the Feature adds something new to the game label Dec 16, 2022
Copy link
Member

@nednaZ nednaZ left a comment

Choose a reason for hiding this comment

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

Minor things, looks fine otherwise

Comment on lines 758 to 759
var/loot = rand(1,4)
switch(loot)
Copy link
Member

Choose a reason for hiding this comment

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

Unneeded variable, just put the rand right in the switch

Copy link
Author

Choose a reason for hiding this comment

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

think i did it now, yet not 100% sure

@dwasint
Copy link
Collaborator

dwasint commented Jan 31, 2023

if we go down this route we might want to look into making them much stronger, as these are currently rather weak in comparison

@Gboster-0 Gboster-0 closed this by deleting the head repository Jan 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Feature adds something new to the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants