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

Explanation for "auto import" is missing #69

Open
froschdesign opened this issue Apr 28, 2022 · 2 comments
Open

Explanation for "auto import" is missing #69

froschdesign opened this issue Apr 28, 2022 · 2 comments
Labels
Bug Something isn't working Documentation

Comments

@froschdesign
Copy link
Member

Q A
Version(s) 0.4.0

The current documentation rule ImplementsFactoryInterfaceToPsrFactoryRector contains the following sentence:

If we want to auto import, we can use LAMINAS_SERVICEMANGER_40_AUTO_IMPORT:

But nowhere is it explained what "auto import" is. An explanation must be added here, preferably the whole part should be rewritten as a separate work step.

If we want to auto import, we can use `LAMINAS_SERVICEMANGER_40_AUTO_IMPORT`:

@froschdesign froschdesign added Bug Something isn't working Documentation labels Apr 28, 2022
@froschdesign
Copy link
Member Author

@samsonasik
Please complete the documentation, otherwise no one knows what the component does and how a developer can use it.
There is a simple rule: what is not documented does not exist.

It would be a pity if your work is not used!

@samsonasik
Copy link
Member

I will try when I have a chance 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants