-
Notifications
You must be signed in to change notification settings - Fork 16
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
The namespace of CSVExtractor seems wrong #2
Labels
Comments
You are right, composer and IDE teamed up to hide this one (tests are working). I'm concerned with BC for this one, have to think. Ty for reporting. |
Composer's autoloader makes it work (unless you use the |
It's not so PSR4 as a namespace, I guess composer could have been a bit more dramatic about it. |
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The namespace of YaEtl/src/Extractors/File/CSVExtractor.php is:
namespace fab2s\YaEtl\Loaders\File;
On the other hand, YaEtl/src/Extractors/File/LineExtractor.php is:
namespace fab2s\YaEtl\Extractors\File;
Seems the first file is in a wrong namespace.
The text was updated successfully, but these errors were encountered: