Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@Absolucy
Copy link
Contributor

@Absolucy Absolucy commented Mar 6, 2022

Document the changes in your pull request

RPEDs can now replace cells in APCs. Hopefully, science/engineering will actually upgrade APCs now.

The new cell is recharged by the old cell, so you can't use this to sabotage stuff by putting an empty bluespace cell in an BRPED and going ham.

Changelog

🆑 Lucy
rcsadd: RPEDs are now capable of changing the power cells in APCs now!
/:cl:

@ToasterBiome
Copy link
Contributor

ToasterBiome commented Mar 6, 2022

Code looks fine, is this locally tested?

This could also be made into a component 👀

@Absolucy
Copy link
Contributor Author

Absolucy commented Mar 7, 2022

@ToasterBiome yes, locally tested. If I hadn't locally tested it, works_with_rped_anyways wouldn't exist as I wouldn't have realized the component_parts requirement as I did during testing :P

@bruhlookatthisdood
Copy link
Contributor

what if you put a fully charged bluespace cell in a brped and use it on an apc? will it be the original cell's charge or the bluespace cell's charge?

@Absolucy
Copy link
Contributor Author

Absolucy commented Mar 7, 2022

@bruhlookatthisdood bluespace cell's charge.

@bruhlookatthisdood
Copy link
Contributor

@bruhlookatthisdood bluespace cell's charge.

thank you

Yogbot-13 added a commit that referenced this pull request Mar 7, 2022
Yogbot-13 added a commit that referenced this pull request Mar 8, 2022
@ToasterBiome ToasterBiome added the TESTMERGED This PR is currently live in a test merge. label Mar 10, 2022
Yogbot-13 added a commit that referenced this pull request Mar 10, 2022
Yogbot-13 added a commit that referenced this pull request Mar 10, 2022
Yogbot-13 added a commit that referenced this pull request Mar 10, 2022
@JamieD1 JamieD1 merged commit 04f494f into yogstation13:master Mar 13, 2022
@Absolucy Absolucy deleted the rped-shit branch March 13, 2022 19:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

TESTMERGED This PR is currently live in a test merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants