Skip to content

optional analysis type check skipping - #1517

Open
jstucke wants to merge 2 commits into
masterfrom
skip-analysis-type-check
Open

optional analysis type check skipping#1517
jstucke wants to merge 2 commits into
masterfrom
skip-analysis-type-check

Conversation

@jstucke

@jstucke jstucke commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator
  • "force_update" was using a field of FileObject that does not "officially" exist -> move field to FileObject.temporary_data
  • add an option for skipping the analysis type check (plugin blacklist/whitelist)
    • can be useful if the file type was detected wrongly and the analysis can't be run otherwise
    • also uses FileObject.temporary_data
    • makes sure FileObject.temporary_data is propagated to included files
    • available for FW update, single file update, and REST analysis PUT

@jstucke
jstucke requested a review from dorpvom July 28, 2026 15:45
@jstucke jstucke self-assigned this Jul 28, 2026
@codecov-commenter

codecov-commenter commented Jul 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.05882% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 92.60%. Comparing base (b796fe7) to head (f98b37a).
⚠️ Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
src/scheduler/analysis/scheduler.py 87.50% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1517      +/-   ##
==========================================
+ Coverage   92.46%   92.60%   +0.14%     
==========================================
  Files         370      368       -2     
  Lines       20828    21306     +478     
==========================================
+ Hits        19259    19731     +472     
- Misses       1569     1575       +6     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

2 participants