Consolidate FrontendServerDdcStrategyProvider and FrontendServerRequireStrategyProvider Description #2517
Labels
P1
A high priority bug; for example, a single project is unusable or has many test failures
package:dwds
triaged
Comments
jyameo
added a commit
to jyameo/webdev
that referenced
this issue
Nov 4, 2024
bkonyi
added
P1
jyameo
added a commit
that referenced
this issue
Nov 12, 2024
…reStrategyProvider under a shared parent class. - [#2517](https://gith… (#2518) * Consolidate and under a shared parent class. - [#2517](#2517) * clean up lib/src/loaders/frontend_server_strategy_provider.dart * updated copyright date, make T extend loadstrategy and define strategies as late * code formatting * combine load strategies into one file and make fields/methods private * updated path to load strategy providers and added a throw statement when new module format (ddc library bundle) is requested
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The classes FrontendServerDdcStrategyProvider and FrontendServerRequireStrategyProvider currently share a lot of common logic and fields. To improve maintainability and reduce redundancy, we should refactor these classes to inherit from a single parent class, FrontendServerStrategyProvider, which will contain shared methods and properties. Each class can then implement only the unique parts required for their respective strategies.
The text was updated successfully, but these errors were encountered: