close
Skip to content

Fixes #527 - Export integers/floats as numbers in ODS format - #528

Merged
hugovk merged 1 commit into
jazzband:masterfrom
claudep:odstypes
Oct 31, 2022
Merged

Fixes #527 - Export integers/floats as numbers in ODS format#528
hugovk merged 1 commit into
jazzband:masterfrom
claudep:odstypes

Conversation

@claudep

@claudep claudep commented Oct 30, 2022

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Oct 30, 2022

Copy link
Copy Markdown

Codecov Report

Merging #528 (f075c18) into master (93f4741) will increase coverage by 0.10%.
The diff coverage is 100.00%.

❗ Current head f075c18 differs from pull request most recent head 4a5ff2e. Consider uploading reports for the commit 4a5ff2e to get more accurate results

@@            Coverage Diff             @@
##           master     #528      +/-   ##
==========================================
+ Coverage   90.84%   90.94%   +0.10%     
==========================================
  Files          28       28              
  Lines        2664     2651      -13     
==========================================
- Hits         2420     2411       -9     
+ Misses        244      240       -4     
Impacted Files Coverage Δ
src/tablib/formats/_ods.py 98.18% <100.00%> (+4.84%) ⬆️
tests/test_tablib.py 98.63% <100.00%> (+0.01%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@matthijskooijman matthijskooijman left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this fix, it seems to work as expected. I left one comment inline to make it even more generic.

Comment thread src/tablib/formats/_ods.py Outdated
Comment thread tests/test_tablib.py Outdated
Comment thread src/tablib/formats/_ods.py Outdated
Comment thread src/tablib/formats/_ods.py Outdated
@hugovk

hugovk commented Oct 31, 2022

Copy link
Copy Markdown
Member

GitHub tip: put "Fixes #527" in the PR message and it will autoclose the issue when merged.

It doesn't trigger from the title.

Thanks Matthijs Kooijman for the report.

Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>

@hugovk hugovk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@hugovk
hugovk merged commit 3617859 into jazzband:master Oct 31, 2022
@claudep
claudep deleted the odstypes branch October 27, 2023 18:31
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.

3 participants