Skip to content

Removing enumerations in favor of class hierarchies#153

Merged
VincenzoArceri merged 5 commits intomasterfrom
enumerations-refactoring
Jan 11, 2022
Merged

Removing enumerations in favor of class hierarchies#153
VincenzoArceri merged 5 commits intomasterfrom
enumerations-refactoring

Conversation

@lucaneg
Copy link
Member

@lucaneg lucaneg commented Jan 11, 2022

Description
Switching from enumerations to interfaces when possible to increase customizability.

Implemented features
Closes #152
Closes #147

@lucaneg lucaneg added the 🛠 type:refactoring Refactoring of the codebase label Jan 11, 2022
@lucaneg lucaneg added this to the 0.1b4 milestone Jan 11, 2022
@lucaneg lucaneg self-assigned this Jan 11, 2022
@lucaneg lucaneg added the 🏗 resolution:wip Incomplete work - do not review yet label Jan 11, 2022
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@lucaneg lucaneg force-pushed the enumerations-refactoring branch from be0f74f to 0a3e0be Compare January 11, 2022 10:47
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@lucaneg lucaneg removed the 🏗 resolution:wip Incomplete work - do not review yet label Jan 11, 2022
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions
Copy link

Unit Test Results

  33 files    33 suites   11s ⏱️
204 tests 204 ✔️ 0 💤 0 ❌

Results for commit 3bd581d.

@github-actions

This comment has been minimized.

@VincenzoArceri VincenzoArceri merged commit b066107 into master Jan 11, 2022
@VincenzoArceri VincenzoArceri deleted the enumerations-refactoring branch January 11, 2022 20:22
@github-actions
Copy link

All Test Results (unit + cron)

  43 files  ±0    43 suites  ±0   15s ⏱️ +2s
226 tests ±0  226 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b066107. ± Comparison against base commit ad5454e.

@lucaneg lucaneg added the 🏨 scope:infrastructure Work regarding the general infrastructure and functioning of LiSA label Jan 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏨 scope:infrastructure Work regarding the general infrastructure and functioning of LiSA 🛠 type:refactoring Refactoring of the codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE REQUEST] Remove enumerations in favor of interfaces [FEATURE REQUEST] Make the ResolutionStrategy more flexible

2 participants