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

Remove spaces from numerics #226

Merged
merged 1 commit into from
Feb 10, 2025
Merged

Remove spaces from numerics #226

merged 1 commit into from
Feb 10, 2025

Conversation

rad-pat
Copy link
Contributor

@rad-pat rad-pat commented Feb 10, 2025

For a number with leading spaces, we were not getting an accurate conversion. Hopefully this helps.

@rad-pat rad-pat requested review from inviscid and chucklay February 10, 2025 17:48
Copy link

File Coverage
All files 39%
plaidcloud/utilities/analyze_table.py 36%
plaidcloud/utilities/bom.py 24%
plaidcloud/utilities/clean_files.py 28%
plaidcloud/utilities/connect.py 35%
plaidcloud/utilities/convert_encoding.py 31%
plaidcloud/utilities/data_helpers.py 26%
plaidcloud/utilities/frame_manager.py 19%
plaidcloud/utilities/pretty_size.py 91%
plaidcloud/utilities/query.py 21%
plaidcloud/utilities/sql_expression.py 85%
plaidcloud/utilities/sqlalchemy_functions.py 63%
plaidcloud/utilities/sqlalchemy_views.py 92%
plaidcloud/utilities/stringtransforms.py 76%
plaidcloud/utilities/udf.py 16%
plaidcloud/utilities/utility.py 19%
plaidcloud/utilities/xray.py 88%
plaidcloud/utilities/remote/dim_helpers.py 10%
plaidcloud/utilities/remote/dimension.py 32%

Minimum allowed coverage is 75%

Generated by 🐒 cobertura-action against b2dbe9e

@rad-pat rad-pat merged commit 7a63bbc into master Feb 10, 2025
4 checks passed
@rad-pat rad-pat deleted the extract-numbers branch February 10, 2025 17:51
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