Skip to content

Conversation

MiniDigger
Copy link
Member

@MiniDigger MiniDigger commented May 5, 2024

doesn't handle multi line commands, kills all whitespace
idk if we care about either

execute as MiniDigger at @s \
    run say hi

@kennytv
Copy link
Member

kennytv commented May 6, 2024

There should also be an additional warning that macros can't be handled and that multi-line commands will be absorbed into a single line

@zlataovce zlataovce added the pre-migration Pull requests which point to a commit before the Astro migration label Apr 20, 2025
@Strokkur424
Copy link
Member

Do we still care about this? I feel like 1.20.5 has been long enough already. We could probably also remove the item converter as a whole, but I think there is no real reason to do that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pre-migration Pull requests which point to a commit before the Astro migration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants