-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BrettDong
added
[C++]
Changes (can be) made in C++. Previously named `Code`
[JSON]
Changes (can be) made in JSON
labels
Feb 22, 2021
ZhilkinSerg
force-pushed
the
0.F-dev
branch
2 times, most recently
from
April 9, 2021 22:14
16fb04b
to
ff86c62
Compare
Ramza13
requested review from
BrettDong,
Fris0uman,
I-am-Erk,
John-Candlebury,
Maleclypse and
Mom-Bun
as code owners
April 20, 2021 06:58
ZhilkinSerg
force-pushed
the
0.F-dev
branch
2 times, most recently
from
April 20, 2021 07:49
592543e
to
54a4e7f
Compare
actual-nh
removed request for
John-Candlebury,
BrettDong,
Fris0uman,
Mom-Bun,
I-am-Erk and
Maleclypse
April 20, 2021 13:53
ZhilkinSerg
force-pushed
the
0.F-dev
branch
3 times, most recently
from
May 31, 2021 07:49
11f8b39
to
a3ed94b
Compare
ZhilkinSerg
force-pushed
the
0.F-dev
branch
3 times, most recently
from
June 14, 2021 15:06
ac1984a
to
c8f13cf
Compare
Ramza13
force-pushed
the
bio_memory
branch
4 times, most recently
from
July 9, 2021 13:12
ad77a6b
to
89b336e
Compare
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
Updates for #47253 done? |
meelock
reviewed
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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be CQB afaik.
3 tasks
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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