-
-
Notifications
You must be signed in to change notification settings - Fork 266
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
Change scope of com.discord4j:stores-api from implementation to api. #1201
Conversation
I'm already here 👻 |
Really appreciate your prompt response :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This don't seem to break neither the compilation nor the use of the lib. However, I'd like to have @quanticc 's opinion on this too (as he added this, maybe there's a reason for why "implementation" over "api").
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine with me. Regarding the reason, it's probably an oversight after moving Store
to the common module, that didn't trigger anything since we use Stores in the context of core module only, where it is configured as api
.
i dont check but this can target 3.2.x? |
Yeah it should, I'll rebase it |
Description: Change scope of com.discord4j:stores-api from implementation to api.
Justification: Fix #1200 according to gradle official doc.