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 Brick Aliases #7579

Closed
wants to merge 1 commit into from
Closed

Conversation

erenkarakal
Copy link
Member

Description

Adds brick and nether brick aliases.

Previous behavior:

brick = doesn't exist
bricks = "bricks banner pattern" - giving it to a player gives brick block
nether brick = doesn't exist
nether bricks = "nether bricks or nether brick" - giving it to a player gives either item randomly
nether brickss = "nether brick" - item

New behavior:

brick = "brick" - item
bricks = "bricks banner pattern" - giving it to a player gives brick block
nether brick = "nether brick" - item
nether bricks = "nether bricks" - block
nether brickss = doesn't exist

This a quick patch, there is a bigger issue that needs to be fixed that will be discussed in a different issue.
With this PR, comparing type of {_block} to bricks doesn't work and is a known issue. Current workaround is comparing it to 1 bricks.


Target Minecraft Versions: any
Requirements: none
Related Issues: #7484

@erenkarakal erenkarakal added the breaking changes Pull or feature requests that contain breaking changes (API, syntax, etc.) label Feb 4, 2025
@sovdeeth
Copy link
Member

sovdeeth commented Feb 4, 2025

This should be a PR on the skript-aliases repo

@erenkarakal erenkarakal closed this Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking changes Pull or feature requests that contain breaking changes (API, syntax, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants