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

four new artifacts #135

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

dwinters99
Copy link
Contributor

@dwinters99 dwinters99 commented Nov 5, 2021

they need code support to be fully implemented but they have icons and buy/sell prices
they work now

@ProbablyCarl
Copy link
Contributor

Having something active as an artifact subclass will cause it to spawn in randomized loot pools. It's why I commented out Nightstar and Stone Flesh temporarily.

@dwinters99 dwinters99 marked this pull request as draft November 10, 2021 13:11
@dwinters99 dwinters99 marked this pull request as ready for review November 17, 2021 17:48
@ProbablyCarl
Copy link
Contributor

I'll give this a lookover now. As you've marked it for review, I'll assume it's ready to be merged if necessary.

/obj/item/artifact/thorn
name = "thorn"
desc = "The result of the interaction between the anomaly Burnt Fuzz and the body of a careless stalker. The Thorn artifact pokes the body of its owner, no matter what. But it also helps clean the body of radionucliodes. Quite widespread and cheap."
eng_desc = "The result of the interaction between the anomaly Burnt Fuzz and the body of a careless stalker. The Thorn artifact pokes the body of its owner, no matter what. But it also helps clean the body of radionucliodes. Quite widespread and cheap."
Copy link
Contributor

Choose a reason for hiding this comment

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

eng_desc is deprecated. Do not use it.

Copy link
Contributor

Choose a reason for hiding this comment

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

Aside from this, the PR is good.
I'll fix this myself if you don't get 'em.

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.

2 participants