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.

Sign up for GitHub

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

Jump to bottom

Refactor: NBT Copying #28

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Refactor: NBT Copying #28

wants to merge 2 commits into from

Conversation

Zickles
Copy link
Contributor

@Zickles Zickles commented Jun 11, 2023

Change Log

  • Remove NBT Command
  • Set nbt copying bind not bound by default
  • Make nbt copying always active

Ascynx
Ascynx previously approved these changes Jun 11, 2023
View reviewed changes
Harry282
Harry282 previously approved these changes Jun 24, 2023
View reviewed changes
@Ascynx Ascynx dismissed stale reviews from Harry282 and themself via d9b02ee June 25, 2023 05:16
Ascynx
Ascynx previously approved these changes Jun 25, 2023
View reviewed changes
@Zickles Zickles dismissed Ascynx’s stale review June 25, 2023 05:17

The merge-base changed after approval.

@Ascynx Ascynx closed this Jun 25, 2023
@Ascynx Ascynx reopened this Jun 25, 2023
@Ascynx Ascynx requested a review from Harry282 June 25, 2023 05:18
@Ascynx Ascynx added the enhancement New feature or request label Jul 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ascynx Ascynx Ascynx approved these changes

@Harry282 Harry282 Awaiting requested review from Harry282

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Zickles @Harry282 @Ascynx