Skip to content
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

Added Parsing to and from CSV #282

Closed
wants to merge 8 commits into from
Closed

Added Parsing to and from CSV #282

wants to merge 8 commits into from

Conversation

Vimal-Raghubir
Copy link

@Vimal-Raghubir Vimal-Raghubir commented Mar 25, 2018

  1. Added operation to parse CSV string to a string. This string is an array of arrays.

  2. Also created an operation that allowed an array of arrays string to be parse to a CSV string.

This is in response to the issue #277.

@d98762625
Copy link
Member

Hey @Vimal-Raghubir thanks for your PR.

I think we may have an opportunity not to use an external library for this piece of functionality (see our principles).

There is already a CSV parsing function in use in src/core/Utils, so that could be the core functionality for one method. For the other half, you could look through the source for the npm library you pulled in, see how they implement it and take inspiration from that?

BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
🌐 Add 🇳🇴 Norwegian translations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants