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 "with all itemflags" to ExprWithItemFlags #7704

Merged
merged 9 commits into from
Mar 21, 2025

Conversation

Fusezion
Copy link
Contributor

@Fusezion Fusezion commented Mar 16, 2025

Description

This PR aims to add support for %itemtype% with all item flags, while initial issue was "bug", it appeared the general consensus was to add with all item flags.


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

Copy link
Member

@sovdeeth sovdeeth left a comment

Choose a reason for hiding this comment

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

maintain parity with patterns

Co-authored-by: sovdee <10354869+sovdeeth@users.noreply.github.com>
@Fusezion Fusezion requested a review from sovdeeth March 16, 2025 00:50
Copy link
Member

@sovdeeth sovdeeth left a comment

Choose a reason for hiding this comment

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

tests failing due to list comparison being in differing order

@sovdeeth sovdeeth added the enhancement Feature request, an issue about something that could be improved, or a PR improving something. label Mar 16, 2025
Co-authored-by: Efnilite <35348263+Efnilite@users.noreply.github.com>
@Fusezion Fusezion requested a review from Efnilite March 16, 2025 12:50
@Efnilite Efnilite added the feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. label Mar 16, 2025
@sovdeeth sovdeeth requested a review from a team as a code owner March 21, 2025 21:25
@sovdeeth sovdeeth requested review from Romitou and UnderscoreTud and removed request for a team March 21, 2025 21:25
@sovdeeth sovdeeth merged commit 7ad27a4 into SkriptLang:dev/feature Mar 21, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature request, an issue about something that could be improved, or a PR improving something. feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants