You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Only the parsers themselves should have to figure out if they can currently support a file extension.
My suggestion:
Add isInstalled to FileParserInterface or alternatively add a new optional interface Installable.
Config can now avoid using parsers that are not currently useable like the YAML parser.
What do you think? The interface would be more backwards compatible I guess, but then you have to check for it each time if it's not part of FileParserInterface.
A new public method
getSupportedFileExtensions
that returns the collected extensions of all parsers.This would make it easier to scan directories only for supported file types.
Edit: If that's too specific, a
getSupportedFileParsers
would be fine too :)I'm happy to implement either, just looking for feedback first.
The text was updated successfully, but these errors were encountered: