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

flecks (tm) #52

Merged
merged 23 commits into from
Sep 23, 2024
Merged

flecks (tm) #52

merged 23 commits into from
Sep 23, 2024

Conversation

Vlue
Copy link
Contributor

@Vlue Vlue commented Sep 14, 2024

rai made me do it

# Conflicts:
#	src/main/java/dev/enjarai/trickster/EndecTomfoolery.java
@Vlue Vlue marked this pull request as draft September 14, 2024 09:52
@enjarai enjarai self-assigned this Sep 14, 2024
@Vlue Vlue marked this pull request as ready for review September 22, 2024 06:39
Copy link
Collaborator

@StellarWitch7 StellarWitch7 left a comment

Choose a reason for hiding this comment

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

Don't forget to load registry classes during mod initialization using an empty function.

@enjarai
Copy link
Owner

enjarai commented Sep 22, 2024

Just woke up, give me a bit to review it as well before merge please! ^^

@StellarWitch7
Copy link
Collaborator

I think @Vlue wanted you to take a look at the UVs.

@StellarWitch7
Copy link
Collaborator

Oh also just noticed that the abstract fleck trick has been misspelled. In addition, I see that the excess throws have been removed. However, there should still be one, as the fleck-making abstract method may blunder.

Copy link
Owner

@enjarai enjarai left a comment

Choose a reason for hiding this comment

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

Needs docs, but other than that, works splendidly!

@enjarai
Copy link
Owner

enjarai commented Sep 22, 2024

I think @Vlue wanted you to take a look at the UVs.

I dont have any issue with the UVs as is. Sure, they're a bit squished near the end of the line, but that isn't an issue imo.

Copy link
Collaborator

@StellarWitch7 StellarWitch7 left a comment

Choose a reason for hiding this comment

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

I may have added something a little silly, for fun.

@StellarWitch7
Copy link
Collaborator

Some conflicts need to be resolved locally, unfortunately.

@StellarWitch7 StellarWitch7 merged commit 340d832 into enjarai:master Sep 23, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants