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
Backend overhaul #111
Backend overhaul #111
Conversation
Wow, BIG change |
src/main/java/io/github/sefiraat/slimetinker/items/componentmaterials/CMManager.java
Outdated
Show resolved
Hide resolved
This is a bit of a massive PR, but I would love you to take a look over it if you have time. If you don't - please do not worry and, if you do, please do take your time, there is no rush :) |
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.
haven't reviewed all
src/main/java/io/github/sefiraat/slimetinker/items/templates/ArmourDefinition.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/templates/ArmourTemplate.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/templates/ToolTemplate.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/templates/ToolTemplateExplosive.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/TinkerMaterial.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/TinkerMaterial.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/TinkerMaterial.java
Outdated
Show resolved
Hide resolved
reviewed 51 files, will review remaining files later |
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.
more reviews
src/main/java/io/github/sefiraat/slimetinker/items/workstations/armourtable/ArmourTable.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/workstations/repairbench/RepairBench.java
Outdated
Show resolved
Hide resolved
...in/java/io/github/sefiraat/slimetinker/items/workstations/smeltery/TinkersSmelteryCache.java
Outdated
Show resolved
Hide resolved
...in/java/io/github/sefiraat/slimetinker/items/workstations/smeltery/TinkersSmelteryCache.java
Outdated
Show resolved
Hide resolved
...in/java/io/github/sefiraat/slimetinker/items/workstations/smeltery/TinkersSmelteryCache.java
Outdated
Show resolved
Hide resolved
...in/java/io/github/sefiraat/slimetinker/items/workstations/smeltery/TinkersSmelteryCache.java
Outdated
Show resolved
Hide resolved
.../java/io/github/sefiraat/slimetinker/items/workstations/swappingstation/SwappingStation.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/workstations/tooltable/ToolTable.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/utils/ItemUtils.java
Outdated
Show resolved
Hide resolved
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.
will review traits when I get up
src/main/java/io/github/sefiraat/slimetinker/items/templates/RepairkitTemplate.java
Outdated
Show resolved
Hide resolved
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.
finished reviewing traits
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/setup/Traits.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/setup/Traits.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/setup/Traits.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/setup/Traits.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/setup/Traits.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/setup/Traits.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/setup/Traits.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/setup/Traits.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/setup/Traits.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/setup/Traits.java
Outdated
Show resolved
Hide resolved
I don't understand why my new reviews are outdated. |
I probably did something dumb :D - I can see them all so this is great, thanks! |
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/setup/Traits.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/sefiraat/slimetinker/items/tinkermaterials/setup/Traits.java
Outdated
Show resolved
Hide resolved
It's ... outdated again. |
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.
LGTM
Nice, ill await the in-live feedback and merge tomorrow |
This is to fix my bad code as much as possible without changing anything breaking.
and anything else to un-dumb.
This will be followed by a second pull to convert all tools to proper PDC Types which will need a much longer lead in time.