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

Unhardcode bio_memory, add several enchantments #47656

Merged
merged 8 commits into from
Jul 10, 2021

Conversation

Ramza13
Copy link
Contributor

@Ramza13 Ramza13 commented Feb 21, 2021

Summary

None

Purpose of change

Unhardcode bio_memory

Describe the solution

Add enchantments/flags for all parts of its effects and use them instead of hardcode. Use enchantments for several traits also having these effects. Remove power usage from preventing skill rust for cleanness of code and because it doesn't make sense that it requires more energy than any of the other usages.

Describe alternatives you've considered

Testing

Additional context

@Ramza13 Ramza13 requested a review from KorGgenT as a code owner February 21, 2021 21:48
@BrettDong BrettDong added [C++] Changes (can be) made in C++. Previously named `Code` [JSON] Changes (can be) made in JSON labels Feb 22, 2021
@BrettDong BrettDong changed the base branch from master to 0.F-dev April 5, 2021 21:57
@ZhilkinSerg ZhilkinSerg force-pushed the 0.F-dev branch 2 times, most recently from 16fb04b to ff86c62 Compare April 9, 2021 22:14
@ZhilkinSerg ZhilkinSerg force-pushed the 0.F-dev branch 2 times, most recently from 592543e to 54a4e7f Compare April 20, 2021 07:49
@ZhilkinSerg ZhilkinSerg force-pushed the 0.F-dev branch 3 times, most recently from 11f8b39 to a3ed94b Compare May 31, 2021 07:49
@ZhilkinSerg ZhilkinSerg force-pushed the 0.F-dev branch 3 times, most recently from ac1984a to c8f13cf Compare June 14, 2021 15:06
@Ramza13 Ramza13 requested a review from LyleSY as a code owner June 21, 2021 22:33
@kevingranade kevingranade changed the base branch from 0.F-dev to master July 5, 2021 06:29
@Ramza13 Ramza13 force-pushed the bio_memory branch 4 times, most recently from ad77a6b to 89b336e Compare July 9, 2021 13:12
@actual-nh actual-nh removed the request for review from LyleSY July 9, 2021 17:07
@actual-nh actual-nh added Bionics CBM (Compact Bionic Modules) Mechanics: Effects / Skills / Stats Effects / Skills / Stats Mechanics: Enchantments / Spells Enchantments and spells and removed 0.F Feature Freeze labels Jul 9, 2021
@actual-nh
Copy link
Contributor

Updates for #47253 done?

@ZhilkinSerg ZhilkinSerg merged commit 43466bd into CleverRaven:master Jul 10, 2021
@@ -1532,4 +1532,4 @@ Gun fault flags:
- ```CLIMATE_CONTROL``` You are resistant to extreme temperatures.
- ```HEATSINK``` You are resistant to extreme heat.
- ```THERMOMETER``` You always know what temperature it is.

- ```CBQ_LEARN_BONUS``` You learn CBQ from the bionic bio_cqb faster.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be CQB afaik.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bionics CBM (Compact Bionic Modules) [C++] Changes (can be) made in C++. Previously named `Code` [JSON] Changes (can be) made in JSON Mechanics: Effects / Skills / Stats Effects / Skills / Stats Mechanics: Enchantments / Spells Enchantments and spells
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants