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

Add some 1.20.5 data #879

Merged
merged 3 commits into from
Jun 23, 2024
Merged

Conversation

qwqtoday
Copy link
Contributor

@qwqtoday qwqtoday commented Jun 9, 2024

1.20.5 update

@qwqtoday qwqtoday mentioned this pull request Jun 9, 2024
@@ -0,0 +1 @@
[]
Copy link
Member

Choose a reason for hiding this comment

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

is this supposed to be blank?

Copy link
Member

Choose a reason for hiding this comment

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

Likely a failure of the extractor, this would need to be fixed. Doesn't seem like there was any changes to 1.20.5 food list so it may be possible to write a script to just update the item IDs.

@@ -0,0 +1,1302 @@
[
Copy link
Member

Choose a reason for hiding this comment

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

ok for now, but there is additional data to extract here with the extractor in tools/js/extractMcpcEntityMetadata for extracting metadata keys (used in mineflayer)

@@ -0,0 +1,58 @@
const fs = require('fs')
Copy link
Member

Choose a reason for hiding this comment

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

I fixed the foods for now with a new extraction script here via the source code

@extremeheat
Copy link
Member

LGTM @rom1504

@extremeheat
Copy link
Member

Going to merge this to not conflict with/block 1.20.5 protocol data

@extremeheat extremeheat changed the title 1.20.5 update Add some 1.20.5 data Jun 23, 2024
@extremeheat extremeheat merged commit 4ca8db1 into PrismarineJS:master Jun 23, 2024
5 checks passed
@rom1504
Copy link
Member

rom1504 commented Jun 24, 2024

Thanks for handling it @extremeheat

@qwqtoday qwqtoday deleted the 1.20.5-update branch June 25, 2024 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Closed
Development

Successfully merging this pull request may close these issues.

3 participants