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

Mining Skillchip #756

Merged
merged 11 commits into from
Jan 16, 2024
Merged

Mining Skillchip #756

merged 11 commits into from
Jan 16, 2024

Conversation

nednaZ
Copy link
Member

@nednaZ nednaZ commented Dec 19, 2023

About The Pull Request

Adds a new neutral (0 point) quirk, "Miner Training", which gives the holder a skillchip allowing them to call out anything mining related they see when they point at it.

It's literally a Deep Rock Galactic reference.
Based off of tgstation/tgstation#72238
Made the suggested changes that contributors had there.

I have very likely missed a number of objects that miners often interact with, but it is very easy to add new ones.

2023-12-19.08-33-45.mp4

Why It's Good For The Game

Meme value aside, this is actually surprisingly more useful than you might think.
Miners are constantly seen as silent and can die without anyone ever realizing it, so giving them a method to easily call out over the radio lets Cargo know if they are alive or not.
And if they go dead silent, they might actually send someone to go searching!

Changelog

🆑
add: Adds "Miner Training" as a Neutral Quirk
/:cl:

The basic functionality of the new Shaft Miner skillchip.
Includes a new list function.
Expands upon the list of pointable entities.
Adds the skillchip to the library vending machine.
Skillchip now a quirk rather than an inherent trait to miners.
Adds more lines.
Updates radio cooldown to be randomized.
@nednaZ
Copy link
Member Author

nednaZ commented Dec 19, 2023

Suggestions very open for lines.
Just leave them here, or directly as file reviews.

@Litberries
Copy link
Contributor

pointing at gold spams we're rich right

@nednaZ
Copy link
Member Author

nednaZ commented Dec 19, 2023

pointing at gold spams we're rich right

/obj/item/stack/ore/gold = list("WE'RE RICH!"), //Yes, yes, you're rich... time to get a move on! I got Management breathing down my neck here!

Mining mob lines are done
More things. I know, helpful!
Finished lines for weapons and materials.
Almost done.
More items, almost done.
I shall finish what couldn't be!
Finally done with these, now it's ready for review.
I found a couple issues testing it.
@nednaZ nednaZ marked this pull request as ready for review December 23, 2023 20:34
Moves the list from a global list to a static skillchip list.

Adds notice spans.
@dwasint dwasint merged commit d08de9e into Monkestation:master Jan 16, 2024
9 of 18 checks passed
github-actions bot added a commit that referenced this pull request Jan 16, 2024
@nednaZ nednaZ deleted the ROCK_AND_STONE_FOREVER branch March 24, 2024 16:57
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.

4 participants