M-Agoumi / M-Agoumi/Simfa

add some way of caching to the injector class

Open
#18 0 comments 0 reactions 1 assignee View on GitHub

@M-Agoumi is already working on this.

Since Sep 12, 2022.

documentation enhancement
Dominant language
PHP
Stars
1
Forks
0
PR merge metrics
No merged PRs in 30d

Description

the injector class right now checks the method each time, making it a bit slow while our purpose is speed, so we should find a way to cache the method parameters, to inject them directly whenever called after the first time, there is two option disable checking, updating the parameters will cause a problem unless the cache has been cleared by the developer, or keep checking if the params and the method are compatible ( take a bit of time) the third option is using both, the first in production where updating methods doesn't happen, second in a development environment. (write documentation while on it too)

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.