Add delimiter override option for CSVs #865

evenreven · 2025-02-18T12:23:15Z

If you want to render values separated with, say, semicolon, this patch lets you do this:

users = User.all

UserExport.new(users, delimiter: ";").call

Since the vast majority will use commas, I've set comma as the default keyword argument, so this will still work:

users = User.all

UserExport.new(users).call

Also added a quickdraw test.

Hope this looks okay! This is my first PR here, so if I'm missing any coding standard, let me know. :)

Useful for semicolon or tab separated values

evenreven added 2 commits February 18, 2025 12:50

Add delimiter option to override commas

f9e3c95

Useful for semicolon or tab separated values

Add CSV test for semicolon separated values

26b5dbc

joeldrapper merged commit 3a74386 into yippee-fun:main Feb 18, 2025
13 checks passed

Provide feedback