Skip to content
This repository was archived by the owner on Nov 1, 2023. It is now read-only.

Fix default windows interceptor list - #3549

Merged
Teo Voinea (tevoinea) merged 5 commits into
microsoft:mainfrom
tevoinea:tevoinea/FixAllowlistExtend
Oct 5, 2023
Merged

Fix default windows interceptor list#3549
Teo Voinea (tevoinea) merged 5 commits into
microsoft:mainfrom
tevoinea:tevoinea/FixAllowlistExtend

Conversation

@tevoinea

Copy link
Copy Markdown
Contributor

Summary of the Pull Request

What is this about?

PR Checklist

  • Applies to work item: #xxx
  • CLA signed. If not, go over here and sign the CLI.
  • Tests added/passed
  • Requires documentation to be updated
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

Comment threadsrc/agent/coverage/src/allowlist.rs Outdated
@tevoinea
Teo Voinea (tevoinea) enabled auto-merge (squash) October 5, 2023 18:08
@codecov-commenter

Codecov Comments Bot (codecov-commenter) commented Oct 5, 2023

Copy link
Copy Markdown

Codecov Report

Merging #3549 (2482a0a) into main (78f9049) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@ Coverage Diff @@## main #3549 +/- ##
==========================================
- Coverage 39.03% 38.97% -0.06% 
==========================================
Files 301 301 Lines 36845 36848 +3 ==========================================
- Hits 14383 14363 -20 - Misses 22462 22485 +23 
FilesCoverage Δ
src/agent/coverage/src/allowlist.rs90.81% <100.00%> (+0.09%)⬆️
src/agent/coverage/src/allowlist/tests.rs100.00% <100.00%> (ø)
...c/agent/onefuzz-task/src/tasks/coverage/generic.rs87.87% <100.00%> (+0.06%)⬆️

... and 4 files with indirect coverage changes

@tevoinea
Teo Voinea (tevoinea) merged commit f9f26b9 into microsoft:mainOct 5, 2023
Adam (AdamL-Microsoft) added a commit that referenced this pull request Oct 5, 2023
This was referenced Oct 5, 2023
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@tevoinea@codecov-commenter@chkeita