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

Prevent shading of commons-lang3 #798

Merged
merged 1 commit into from
May 3, 2024

Conversation

fionera
Copy link

@fionera fionera commented Apr 2, 2024

I didn't found anything regarding contribution guidelines so I hope this is fine 😄


commons-lang3 is always provided at runtime,
don't shade it to prevent incompatibilities.

Closes #617, #616

commons-lang3 is always provided at runtime,
don't shade it to prevent incompatibilities.

Closes ValkyrienSkies#617, ValkyrienSkies#616
@CLAassistant
Copy link

CLAassistant commented Apr 2, 2024

CLA assistant check
All committers have signed the CLA.

@ewoudje
Copy link
Contributor

ewoudje commented May 3, 2024

A bit late since i kinda wanted to include this fix in #800 but it seems thats not happening soon so merging it!!

@ewoudje ewoudje merged commit 682a6e8 into ValkyrienSkies:1.20.1/main May 3, 2024
5 checks passed
@ewoudje
Copy link
Contributor

ewoudje commented May 3, 2024

And ofc thanks for ur contribution

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