I added the custom extractor tap-sftp to my projec...
# announcements
i
I added the custom extractor tap-sftp to my project. In order to get a CSV file to load which has no quotes but ' as part of some fields I needed to extend the following singer file...singer_encodings/csv.py. My question is now where to address this change to make sure that a meltano update won't break this fix. Is meltano working on a singer.io fork or would I need to communicate with the guys from singer...actually this file could be extended even more to give a better support to the various CSV dialects that are supported by the csv.DictReader. What would be the best way forward?
d
@ingo_klose We have no plans to fork https://github.com/singer-io/tap-sftp, so if you've made a change you'll want to fork it yourself and contribute the change back to that repo. In the mean time, you can use your own fork in your Meltano project: http://meltano.com/docs/plugin-management.html#using-a-custom-fork-of-a-plugin Note that https://meltano.com/plugins/extractors/spreadsheets-anywhere.html also supports SFTP and CSV, and may already handle the quotes/encoding better.
i
OK, thanks @douwe_maan