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

U.M.E.F. away mission #620

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

Conversation

Wallemations
Copy link
Contributor

image

𝓜𝔂 𝓓𝓮𝓪𝓻𝓮𝓼𝓽 𝓜𝓮𝓵𝓫𝓮𝓻𝓽,

𝓘 𝓪𝓶 𝔀𝓻𝓲𝓽𝓲𝓷𝓰 𝓽𝓸 𝔂𝓸𝓾 𝓽𝓸𝓭𝓪𝔂 𝓱𝓾𝓶𝓫𝓵𝔂 𝓻𝓮𝓺𝓾𝓮𝓼𝓽𝓲𝓷𝓰 𝓸𝓯 𝔂𝓸𝓾 𝓽𝓸 𝓰𝓪𝔃𝓮 𝓽𝓱𝓲𝓷𝓮 𝓮𝔂𝓮𝓼 𝓾𝓹𝓸𝓷 𝓶𝔂 𝓱𝓾𝓶𝓫𝓵𝓮 𝓟𝓡. 𝓘𝓯 𝔂𝓸𝓾 𝓭𝓸 𝓷𝓸𝓽 𝓘 𝔀𝓲𝓵𝓵 𝓫𝓮 𝓯𝓸𝓻𝓬𝓮𝓭 𝓽𝓸 𝓼𝓮𝓮𝓴 𝓪𝓵𝓽𝓮𝓻𝓷𝓪𝓽𝓮 𝓶𝓮𝓪𝓷𝓼 𝓸𝓯 𝓱𝓪𝓿𝓲𝓷𝓰 𝓶𝔂 𝓟𝓡 𝓻𝓮𝓿𝓲𝓮𝔀𝓮𝓭, 𝓵𝓸𝔀𝓮𝓻𝓲𝓷𝓰 𝔂𝓸𝓾𝓻 𝓖𝓲𝓽𝓱𝓾𝓫 𝓪𝓬𝓽𝓲𝓿𝓲𝓽𝔂 𝓪𝓷𝓭 𝓱𝓲𝓷𝓭𝓮𝓻𝓲𝓷𝓰 𝔂𝓸𝓾𝓻 𝓯𝓾𝓽𝓾𝓻𝓮 𝓬𝓱𝓪𝓷𝓬𝓮𝓼 𝓸𝓯 𝓾𝓽𝓲𝓵𝓲𝔃𝓲𝓷𝓰 𝓲𝓽 𝓪𝓼 𝓹𝓪𝓻𝓽 𝓸𝓯 𝔂𝓸𝓾𝓻 𝓻𝓮𝓼𝓾𝓶𝓮 𝔀𝓱𝓲𝓬𝓱 𝓘'𝓶 𝓼𝓾𝓻𝓮 𝔂𝓸𝓾 𝔀𝓸𝓾𝓵𝓭 𝓯𝓲𝓷𝓭 𝓶𝓸𝓼𝓽 𝓭𝓲𝓼𝓻𝓾𝓹𝓽𝓲𝓿𝓮.

𝓓𝓸 𝓶𝓪𝓴𝓮 𝓱𝓪𝓼𝓽𝓮 𝓲𝓷 𝓽𝓱𝓮𝓼𝓮 𝓶𝓪𝓽𝓽𝓮𝓻𝓼 𝓪𝓼 𝓽𝓲𝓶𝓮 𝓲𝓼 𝓸𝓯 𝓽𝓱𝓮 𝓾𝓽𝓶𝓸𝓼𝓽 𝓮𝓼𝓼𝓮𝓷𝓼𝓮.

𝓨𝓸𝓾𝓻𝓼 𝓕𝓸𝓻𝓮𝓿𝓮𝓻,
𝓐𝓰𝓮𝓷𝓽 𝓖𝓻𝓮𝓮𝓷

/obj/machinery/computer/slot_machine/green/Initialize(mapload)
. = ..()
name = pick(possible_names)
money = rand(500,1000000)
Copy link
Contributor

Choose a reason for hiding this comment

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

i feel like this should round to the nearest 100

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that's not as funny

Copy link
Owner

@MrMelbert MrMelbert left a comment

Choose a reason for hiding this comment

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

I was originally going to suggest that all related code files (dm, dmi, oggs) be put into one prime folder, ie "the casino subfolder", but I'm tossing up whether or not it's worth while. Basically do it if you want, if you're lazy then it's whatever. One comment though

Comment on lines 1 to 3
// Sources of status effects
// The casino's maze
#define CASINO_MAZE "casino_maze"
Copy link
Owner

Choose a reason for hiding this comment

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

This doesn't need to be in its own file, it's only used in two functions

Just def it before maze/Entered and undef it after maze/Exited

Copy link
Contributor Author

Choose a reason for hiding this comment

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

aight

@Wallemations
Copy link
Contributor Author

I was originally going to suggest that all related code files (dm, dmi, oggs) be put into one prime folder, ie "the casino subfolder", but I'm tossing up whether or not it's worth while. Basically do it if you want, if you're lazy then it's whatever. One comment though

I tried to at least group some of the stuff together, like with the modules/casino folder

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