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

Specify all 3rdparty dependencies in the root build.gradle file #345

Open
sundargates opened this issue Feb 26, 2023 · 4 comments · May be fixed by #349
Open

Specify all 3rdparty dependencies in the root build.gradle file #345

sundargates opened this issue Feb 26, 2023 · 4 comments · May be fixed by #349
Assignees
Labels
good first issue Good for newcomers

Comments

@sundargates
Copy link
Contributor

Context

Mantis has several sub-projects, and every sub-project depends on several 3rd party dependencies. Therefore, we would like consistency around how the 3rd party dependencies are defined and their versions. This would ensure a consistent set of jars in the classpath irrespective of the project that's getting deployed.

To achieve this goal, we introduced a new format for defining 3rd party dependencies in subprojects, as shown below.

implementation libraries.slf4jApi (correct)
implementation "org.slf4j:slf4j-api:${versions.slf4j}" (wrong)
implementation "org.slf4j:slf4j-api:1.2.3" (wrong)

Corresponding root.gradle changes:

ext.libraries = [
  slf4jApi       : "org.slf4j:slf4j-api:${versions.slf4j}"
]

This type of definition achieves the goal defined earlier. However, it is yet to be fully adopted. This ticket aims to do this cleanup for every subproject within mantis and create one or more PRs.

If you will work on this ticket, please create multiple PRs or have multiple commits for every subproject you have refactored to ensure it's easy to review the code.

@sundargates sundargates added the good first issue Good for newcomers label Feb 26, 2023
@mabelbot
Copy link

Hi @sundargates! Thank you for this context, I have read through it and currently working through making these changes in each subproject. I'll keep an eye on mantis-api when #340 is resolved as well if necessary and update here if I have any further questions.

@yugborana
Copy link

hey @sundargates may be I can help in this if you can guide me more on the process. I have to change the code of dependencies or anything else? I am new to this

@sundargates
Copy link
Contributor Author

@mabelbot Do you want me to assign this ticket to you? I would prefer that we do these changes module by module. However, since mantis-api has yet to be merged, you can ignore that PR for now.

@mabelbot
Copy link

@sundargates Yes, sure! We are on the same page w/ doing the changes module by module. It seems some might already be fully with the new format so I've started with ones that are only partially done. I'll chat with you on Discord about this & that tutorial build issue I ran into. Thanks!

mabelbot added a commit to mabelbot/mantis that referenced this issue Feb 28, 2023
chore: update 3rd party dependency specifications for mantis-common and corresponding changes to root build.gradle (issue Netflix#345)
@mabelbot mabelbot linked a pull request Feb 28, 2023 that will close this issue
38 tasks
mabelbot added a commit to mabelbot/mantis that referenced this issue Feb 28, 2023
chore: update netty-transport-native-epoll dependency specification to libraries.nettyEpoll in mantis-common and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Feb 28, 2023
refactor mantis-common build.gradle by moving ext versions to root build.gradle and removing ext variable (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 1, 2023
first pass at refactor 3rd party dependency specifications for mantis-connector-iceberg and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 1, 2023
first pass at refactor 3rd party dependency specifications for mantis-connector-job and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 1, 2023
first pass at refactor 3rd party dependency specifications for mantis-connector-kafka and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 1, 2023
first pass at refactor 3rd party dependency specifications for mantis-control-plane-core and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 1, 2023
first pass at refactor 3rd party dependency specifications for mantis-control-plane-server and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 1, 2023
refactor affects mantis-connector-iceberg and mantis-control-plane-server build.gradle files and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
first pass at refactor 3rd party dependency specifications for mantis-network and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
first pass at refactor 3rd party dependency specifications for mantis-publish-core and corresponding changes to root build.gradle. Name changes to make version vars consistent. (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
first pass at refactor 3rd party dependency specifications for mantis-publish-netty and corresponding changes to root build.gradle. tbd about shadowjar (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
first pass at refactor 3rd party dependency specifications for mantis-publish-netty-guice and corresponding changes to root build.gradle. (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
first pass at refactor 3rd party dependency specifications for mantis-remote-observable and corresponding changes to root build.gradle. (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
first pass at refactor 3rd party dependency specifications for mantis-server-worker and corresponding changes to root build.gradle. tbd for lines 31,32 - exclude (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
first pass at refactor 3rd party dependency specifications for mantis-server-worker-client and corresponding changes to root build.gradle. (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
first pass at refactor 3rd party dependency specifications for mantis-shaded and corresponding changes to root build.gradle. tbd - issue Netflix#194 and grouping (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
first pass at refactor 3rd party dependency specifications for mantis-source-job-kafka. (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
…ations

first pass at refactor 3rd party dependency specifications for mantis-examples-groupby-sample. (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
…cifications

first pass at refactor 3rd party dependency specifications for mantis-examples-jobconnector-sample and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
…pecifications

first pass at refactor 3rd party dependency specifications for mantis-examples-mantis-publish-sample and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
…cy specifications

first pass at refactor 3rd party dependency specifications for mantis-examples-mantis-publish-web-sample and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
…tions

first pass at refactor 3rd party dependency specifications for mantis-examples-sine-function and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
…cifications

first pass at refactor 3rd party dependency specifications for mantis-examples-synthetic-sourcejob and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
…ations

first pass at refactor 3rd party dependency specifications for mantis-examples-twitter-sample and corresponding changes to root build.gradle (issue Netflix#345)
mabelbot added a commit to mabelbot/mantis that referenced this issue Mar 2, 2023
first pass at refactor 3rd party dependency specifications for mantis-examples-wordcount and corresponding changes to root build.gradle (issue Netflix#345)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants