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

Import changedSlots computation from prismarine-windows #3134

Merged
merged 9 commits into from
Aug 5, 2023
Merged

Import changedSlots computation from prismarine-windows #3134

merged 9 commits into from
Aug 5, 2023

Conversation

kaduvert
Copy link
Contributor

@kaduvert kaduvert commented Jul 30, 2023

moves the slot-comparing calculation of changedSlots here because it's of no use in prismarine-windows
the clicks can still return changedSlots directly to save computation. some of them do

depends on PrismarineJS/prismarine-windows#102 for ci

@kaduvert kaduvert changed the title Changed slots import from prismarine-windows Import changedSlots computation from prismarine-windows Jul 30, 2023
@rom1504
Copy link
Member

rom1504 commented Aug 5, 2023

CI fails

@kaduvert
Copy link
Contributor Author

kaduvert commented Aug 5, 2023

CI fails

yes, because

depends on PrismarineJS/prismarine-windows#102 for ci

@kaduvert
Copy link
Contributor Author

kaduvert commented Aug 5, 2023

added the primsarine-windows pr as dependency and fixed some minor bugs.
ci passes @rom1504

@rom1504 rom1504 merged commit e5b5eee into PrismarineJS:master Aug 5, 2023
@rom1504
Copy link
Member

rom1504 commented Aug 5, 2023

ok thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants