-
-
Notifications
You must be signed in to change notification settings - Fork 211
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
Advancements implementation #1645
Advancements implementation #1645
Conversation
f217be9
to
1f83ce7
Compare
@@ -66,13 +76,13 @@ | |||
|
|||
@Shadow @Nullable public Advancement parent; | |||
@Shadow @Final private ResourceLocation id; | |||
@Shadow @Final private String[][] requirements; |
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.
Are these no longer final? Or are you mutating them later with the added AT? In all honesty, you could still set them internally because @Mutable
and @Final
still allows you to set these values within code. The only time you realistically have to mark them as non final is if you want to make a method overridable. Unless I'm mistaken and @Mumfrey can correct me on the statement above.
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.
I am not very familiar with all the available features in mixin, that's why I used the AT. I replaced most of the AT entries with @Mutable
fields in the latest commit.
4fd2255
to
5e12b9a
Compare
…ated in JSON using a extra trigger_times entry.
512ca54
to
b917de7
Compare
Signed-off-by: Gabriel Harris-Rouquette <gabizou@me.com>
API | Common | Forge | Vanilla
This is the implementation of the advancements API.