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

Type hint arch/bpf #3825

Merged
merged 2 commits into from
Nov 28, 2023
Merged

Type hint arch/bpf #3825

merged 2 commits into from
Nov 28, 2023

Conversation

gpotter2
Copy link
Member

This PR adds:

  • typing of arch/bpf -> core fully typed

This will not make it into 2.5.0 but was technically ready before 2.5.0 so the claim that we typed the core is valid 😋

@gpotter2 gpotter2 added the Hinty label Dec 24, 2022
@codecov
Copy link

codecov bot commented Dec 24, 2022

Codecov Report

Merging #3825 (bc8e58d) into master (5a2dbf0) will decrease coverage by 0.70%.
The diff coverage is 1.69%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3825      +/-   ##
==========================================
- Coverage   81.95%   81.25%   -0.70%     
==========================================
  Files         331      331              
  Lines       76647    76654       +7     
==========================================
- Hits        62813    62284     -529     
- Misses      13834    14370     +536     
Files Coverage Δ
scapy/arch/linux.py 85.30% <50.00%> (-1.29%) ⬇️
scapy/arch/bpf/consts.py 0.00% <0.00%> (-100.00%) ⬇️
scapy/arch/bpf/core.py 0.00% <0.00%> (-87.13%) ⬇️
scapy/arch/bpf/supersocket.py 0.00% <0.00%> (-71.22%) ⬇️

... and 13 files with indirect coverage changes

@gpotter2
Copy link
Member Author

This should have been merged a year ago.

@gpotter2 gpotter2 merged commit 3eee39e into secdev:master Nov 28, 2023
20 of 21 checks passed
@gpotter2 gpotter2 deleted the macos branch November 28, 2023 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant