Skip to content

Commit

Permalink
docs: Added a TODO, to refactor the strategy mecahnism
Browse files Browse the repository at this point in the history
This should be refactored to be a switch statement probably or better
yet have a map of strategies and allow the user to set the one they want
to use. But I fear I will break backwards compatibility if I do that.

Signed-off-by: Stefan Genov <stefan.genov@ing.com>
  • Loading branch information
gu74tu authored and Michaelpalacce committed May 16, 2024
1 parent 664389c commit fab82f6
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ public static <T extends Configuration> PackageStore<?> getInstance(T configurat
String version;

// @TODO: You should be able to select a strategy to use, this doesn't make much sense,
// we can fix it later
// but we have no choice since we want to be backward compatible
if (configuration.isForceImportLatestVersions()) {
logger.info("Using StrategyForceLatestVersions");
strategies.add(new StrategyForceLatestVersions());
Expand Down

0 comments on commit fab82f6

Please sign in to comment.