Skip to content

Expand on instructions and link for the README #273

Closed
amishn2008 wants to merge 4 commits intomgastner:mainfrom
amishn2008:requirements
Closed

Expand on instructions and link for the README #273
amishn2008 wants to merge 4 commits intomgastner:mainfrom
amishn2008:requirements

Conversation

@amishn2008
Copy link
Contributor

Added extra instructions for navigation on the README

@amishn2008
Copy link
Contributor Author

@adisidev

@nihalzp
Copy link
Collaborator

nihalzp commented Jun 8, 2025

Thank you for working on adding support for scientific notation.

src/misc/string_to_decimal_converter.cpp is an error-prone part of the software. Please add the tests for the new cases to: tests/unit/test_string_to_decimal_converter.cpp

@nihalzp nihalzp added the feature New feature or request label Jun 8, 2025
@nihalzp nihalzp linked an issue Jun 8, 2025 that may be closed by this pull request
@amishn2008
Copy link
Contributor Author

amishn2008 commented Jun 12, 2025

Moved to #282

@amishn2008 amishn2008 closed this Jun 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow scientific notation for CSV files

3 participants