Skip to content

Conversation

@Calvinnix
Copy link
Collaborator

No description provided.

@Calvinnix Calvinnix requested a review from a team as a code owner July 21, 2025 15:04
@Calvinnix Calvinnix requested a review from andrewzlchen July 21, 2025 15:04
Copy link
Collaborator

@andrewzlchen andrewzlchen left a comment

Choose a reason for hiding this comment

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

lgtm!

*/
tasks.register<ShadowJar>("confluentJar") {
archiveClassifier.set("confluent")
from(mongoDependencies, sourceSets.main.get().output)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should we remove mongoDependencies from the rest of the codebase?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yeah good idea, we can always re-add it if we want to go back to not packaging avro for the confluent jar.

@Calvinnix Calvinnix merged commit f947a07 into mongodb:master Jul 21, 2025
16 of 18 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.

3 participants