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

Item list things and forge!!!!!! #948

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Item list things and forge!!!!!! #948

wants to merge 6 commits into from

Conversation

viciscat
Copy link
Collaborator

@viciscat viciscat commented Aug 20, 2024 •
edited
Loading

  • Make recipes mod independent
  • Forge yipee
  • Did not do JEI cuz what in the hell is hapenning in there

There's probably some cleaning to do but i'm so done with this gaaahhh

@LifeIsAParadox LifeIsAParadox added the reviews needed This PR needs reviews label Aug 20, 2024
@AzureAaron AzureAaron added the new feature This issue or PR is a new feature label Aug 22, 2024
@AzureAaron AzureAaron added this to the 1.23.0 milestone Aug 22, 2024
kevinthegreat1
kevinthegreat1 requested changes Sep 8, 2024
View reviewed changes
Copy link
Collaborator

@kevinthegreat1 kevinthegreat1 left a comment

Choose a reason for hiding this comment

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

I added forge duration with 203d4a2. You might want to look at that. Also what's going on with CreeperEntityAccessor? Other than that looks good.

build.gradle Outdated Show resolved Hide resolved
src/main/java/de/hysky/skyblocker/mixins/CreeperEntityAccessor.java Outdated Show resolved Hide resolved
src/main/resources/skyblocker.mixins.json Outdated Show resolved Hide resolved
@LifeIsAParadox LifeIsAParadox added changes requested This PR need changes and removed reviews needed This PR needs reviews labels Sep 8, 2024
@LifeIsAParadox LifeIsAParadox added reviews needed This PR needs reviews and removed changes requested This PR need changes labels Sep 17, 2024
kevinthegreat1
kevinthegreat1 approved these changes Sep 17, 2024
View reviewed changes
Copy link
Collaborator

@kevinthegreat1 kevinthegreat1 left a comment

Choose a reason for hiding this comment

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

Not tested.

@LifeIsAParadox LifeIsAParadox added merge me please Pull requests that are ready to merge and removed reviews needed This PR needs reviews labels Sep 17, 2024
@AzureAaron AzureAaron removed the merge me please Pull requests that are ready to merge label Oct 12, 2024
@AzureAaron
Copy link
Collaborator

Removed merge me label because half of this just won't work with 1.21.2.

@kevinthegreat1 kevinthegreat1 added the merge conflicts This PR has merge conflicts that need solving. label Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kevinthegreat1 kevinthegreat1 kevinthegreat1 approved these changes

Assignees
No one assigned
Labels
merge conflicts This PR has merge conflicts that need solving. new feature This issue or PR is a new feature
Projects
None yet
Milestone
1.23.0
Development

Successfully merging this pull request may close these issues.
4 participants
@viciscat @AzureAaron @kevinthegreat1 @LifeIsAParadox