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

Add plugin functional, initializers loader, additional methods to monads, refactor code, remove ReturnEngine class #12

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

AnotherRegularDude
Copy link
Member

@AnotherRegularDude AnotherRegularDude commented Dec 14, 2024

Changes

  • Added the ability to modify the base service class through a plugin system like in the Sequel
  • Implement the functionality of applying different initializers to service classes.
    If a parent class has already applied one of the initializers,
    then an attempt to reapply initializer will cause an error for all of his children.
    Therefore, I left the basic service class Resol::Service without the initializer at all, this is breaking behavior.
    You need something like ApplicationSerivce < Resol::Service
  • Remove all methods from Resol::Result, because Result specific methods logic is more optimal.
    Also, add #bind and #fmap methods.
  • Remove ReturnEngine and add plugin instead.
  • Add dependency container from `dry`` lib. Also use config lib from them.

@AnotherRegularDude AnotherRegularDude force-pushed the feature/add-dry-initializer branch from e3b0c0c to 52567c4 Compare January 11, 2025 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant