Skip to content

Conversation

@Absolutionism
Copy link
Contributor

@Absolutionism Absolutionism commented Dec 18, 2024

Description

This PR aims to add additional brewing support such as events, expressions and slots. Allowing users to use a specific expression to get a specific slot of a brewing stand inventory, brewing stand events, and values such as brewing time and brewing fuel.


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

…vents

# Conflicts:
#	src/main/java/ch/njol/skript/classes/data/BukkitEventValues.java
#	src/main/java/ch/njol/skript/events/SimpleEvents.java
@Efnilite Efnilite added the enhancement Feature request, an issue about something that could be improved, or a PR improving something. label Dec 18, 2024
…vents

# Conflicts:
#	src/main/java/ch/njol/skript/classes/data/BukkitEventValues.java
#	src/main/java/ch/njol/skript/events/SimpleEvents.java
@Absolutionism Absolutionism requested a review from Romitou January 13, 2025 20:25
@Absolutionism Absolutionism requested a review from a team as a code owner March 19, 2025 22:27
@Absolutionism Absolutionism requested a review from sovdeeth March 19, 2025 22:27
@Efnilite Efnilite self-requested a review March 20, 2025 00:23
…vents

# Conflicts:
#	src/main/java/ch/njol/skript/classes/data/BukkitEventValues.java
#	src/main/java/ch/njol/skript/events/SimpleEvents.java
…vents

# Conflicts:
#	src/main/java/ch/njol/skript/events/EvtItem.java
@Absolutionism Absolutionism moved this from In Progress to In Review in 2.13 Releases Jul 4, 2025
Absolutionism and others added 2 commits July 28, 2025 12:47
Co-authored-by: Efnilite <35348263+Efnilite@users.noreply.github.com>
@Absolutionism Absolutionism requested a review from Efnilite July 28, 2025 16:47
Copy link
Member

@APickledWalrus APickledWalrus left a comment

Choose a reason for hiding this comment

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

I think it would be good to put this into a module and standardize some of the verb effects "make", "force", "prevent", "block", etc.

Not required for this to be merged though.

@github-project-automation github-project-automation bot moved this from In Review to Awaiting Merge in 2.13 Releases Sep 30, 2025
@Absolutionism Absolutionism requested a review from a team as a code owner October 1, 2025 18:16
@Absolutionism Absolutionism requested review from UnderscoreTud and removed request for a team October 1, 2025 18:16
@Absolutionism Absolutionism merged commit 0d33fdd into SkriptLang:dev/feature Oct 1, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from Awaiting Merge to Done - Awaiting Release in 2.13 Releases Oct 1, 2025
@skriptlang-automation skriptlang-automation bot added the completed The issue has been fully resolved and the change will be in the next Skript update. label Oct 1, 2025
@sovdeeth sovdeeth moved this from Done - Awaiting Release to Done - Released in 2.13 Releases Oct 15, 2025
erenkarakal pushed a commit to erenkarakal/Skript that referenced this pull request Nov 26, 2025
* Initial Commit

* Tests

* Fix JUnit 17

* Requested Changes

* Changes

* Update EventValues

* Requested Changes

* Update Test

* Requested Changes

* Update EvtItem.java

* Partial Changes

* Update EvtItem.java

* Brewing Results Changer

* Update ExprBrewingTime.java

* Changes

* Pickle's Changes

* Requested Changes

* Apply suggestions from code review

Co-authored-by: Efnilite <35348263+Efnilite@users.noreply.github.com>

* Module

* Update SimpleEvents.java
erenkarakal pushed a commit to erenkarakal/Skript that referenced this pull request Nov 26, 2025
* Initial Commit

* Tests

* Fix JUnit 17

* Requested Changes

* Changes

* Update EventValues

* Requested Changes

* Update Test

* Requested Changes

* Update EvtItem.java

* Partial Changes

* Update EvtItem.java

* Brewing Results Changer

* Update ExprBrewingTime.java

* Changes

* Pickle's Changes

* Requested Changes

* Apply suggestions from code review

Co-authored-by: Efnilite <35348263+Efnilite@users.noreply.github.com>

* Module

* Update SimpleEvents.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

completed The issue has been fully resolved and the change will be in the next Skript update. enhancement Feature request, an issue about something that could be improved, or a PR improving something.

Projects

No open projects
Status: Done - Released

Development

Successfully merging this pull request may close these issues.

Add BrewEvent

7 participants