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

Update data_files.rst #2444

Merged
merged 4 commits into from
Jul 14, 2023
Merged

Update data_files.rst #2444

merged 4 commits into from
Jul 14, 2023

Conversation

dagewa
Copy link
Member

@dagewa dagewa commented Jun 22, 2023

No longer pickle

@codecov
Copy link

codecov bot commented Jun 22, 2023

Codecov Report

Merging #2444 (b3bd34c) into main (d92512c) will decrease coverage by 0.01%.
The diff coverage is n/a.

❗ Current head b3bd34c differs from pull request most recent head 8f271d3. Consider uploading reports for the commit 8f271d3 to get more accurate results

@@            Coverage Diff             @@
##             main    #2444      +/-   ##
==========================================
- Coverage   78.71%   78.71%   -0.01%     
==========================================
  Files         606      606              
  Lines       74078    74078              
  Branches    10067    10067              
==========================================
- Hits        58311    58307       -4     
- Misses      13629    13631       +2     
- Partials     2138     2140       +2     

@phyy-nx
Copy link
Member

phyy-nx commented Jun 22, 2023

Maybe include a code snippet on how to open them? Just thinking of how to help folks not familiar with MessagePack.

Add code examples
@dagewa dagewa merged commit e5dcb34 into main Jul 14, 2023
16 checks passed
@dagewa dagewa deleted the dagewa-patch-1 branch July 14, 2023 15:09
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.

None yet

3 participants