https://meltano.com/ logo
#announcements
Title
# announcements
c

clever-football-96790

02/01/2021, 2:44 PM
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?
1
r

ripe-musician-59933

02/01/2021, 3:51 PM
@clever-football-96790 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.
c

clever-football-96790

02/02/2021, 5:51 AM
OK, thanks @ripe-musician-59933