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

Extensions Library Modularization #84

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

alexjhawk
Copy link
Collaborator

@alexjhawk alexjhawk commented Nov 17, 2023

While this project may introduce significant variations between the Maven environment provided by the starter project, the goal with modularization is to maintain our existing library code but make it lighter.

By modularizing, connectors can selectively import the required library modules without the overhead of including the remaining modules (both during runtime and in overall Jar size).

It is expected that this PR will cause several build failures. These will be addressed/resolved using upgrades to the automation to support the modular Maven project structure. In addition, the release documentation will be updated to describe the necessary information for each module to be installed/imported. This will likely result in the usage of JitPack for distribution. While our current Maven repository structure remains operational, the number of new/separate Jars and files associated with modularization will rapidly make it messy and difficult to manage. JitPack already runs on its own for all public GitHub repos, so there is little to no effort on our end to switch to this form of distribution for the extensions library.

Edit: See https://jitpack.io/#hms-networks/sc-ewon-flexy-extensions-lib/dev~2.0.0-modular-SNAPSHOT for testing

@alexjhawk alexjhawk added the enhancement New feature or request label Nov 17, 2023
@alexjhawk alexjhawk self-assigned this Nov 17, 2023
@alexjhawk alexjhawk force-pushed the dev/2.0.0-modular branch 3 times, most recently from 30bc678 to ad63997 Compare November 17, 2023 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant