Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix zip slip #2634

Merged
merged 1 commit into from
Sep 30, 2023
Merged

fix zip slip #2634

merged 1 commit into from
Sep 30, 2023

Conversation

@msaroufim msaroufim changed the title fix zip slip error fix zip slip with model archiver Sep 29, 2023
@msaroufim msaroufim changed the title fix zip slip with model archiver fix zip slip Sep 29, 2023
@codecov
Copy link

codecov bot commented Sep 29, 2023

Codecov Report

Merging #2634 (b76e40d) into master (d8a1146) will decrease coverage by 0.11%.
Report is 3 commits behind head on master.
The diff coverage is n/a.

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

@@            Coverage Diff             @@
##           master    #2634      +/-   ##
==========================================
- Coverage   71.34%   71.24%   -0.11%     
==========================================
  Files          85       85              
  Lines        3905     3905              
  Branches       58       58              
==========================================
- Hits         2786     2782       -4     
- Misses       1115     1119       +4     
  Partials        4        4              

see 2 files with indirect coverage changes

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

@msaroufim msaroufim added this pull request to the merge queue Sep 29, 2023
Merged via the queue into master with commit bfb3d42 Sep 30, 2023
10 of 12 checks passed
@msaroufim msaroufim deleted the msaroufim/zipslip branch September 30, 2023 19:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants