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

Template fix #102

Merged
merged 10 commits into from
Aug 20, 2024
Merged

Template fix #102

merged 10 commits into from
Aug 20, 2024

Conversation

JaaiDead
Copy link
Contributor

im not sure but this might be the fix for it?

@JaaiDead
Copy link
Contributor Author

JaaiDead commented Jul 31, 2024

public static final String MOD_ID = "modid"
public static final Logger LOGGER = LoggerFactory.getLogger(MOD_ID);
Trying to add this

@JaaiDead JaaiDead requested a review from Octol1ttle July 31, 2024 15:41
@dicedpixels
Copy link

Shouldn't the MOD_ID be passed to getLogger?

@JaaiDead
Copy link
Contributor Author

JaaiDead commented Aug 5, 2024

Fixed In Indentation Test i suppose its needs to be public static final Logger LOGGER = LoggerFactory.getLogger(MOD_ID) ?

@JaaiDead JaaiDead requested a review from Octol1ttle August 5, 2024 14:06
@JaaiDead
Copy link
Contributor Author

JaaiDead commented Aug 5, 2024

Can you give me an example to how it should look like ?

@JaaiDead
Copy link
Contributor Author

JaaiDead commented Aug 5, 2024

image
If this is alr PR can be merged

@JaaiDead JaaiDead requested a review from modmuss50 August 5, 2024 15:00
@JaaiDead JaaiDead requested a review from Octol1ttle August 7, 2024 16:33
@JaaiDead
Copy link
Contributor Author

@Octol1ttle For Review:D

@modmuss50
Copy link
Member

Ill merge this now, sorry it took me so long ive been busy with other things.

@modmuss50 modmuss50 merged commit cae4682 into FabricMC:main Aug 20, 2024
2 checks 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.

4 participants