Skip to content
This repository was archived by the owner on Jun 13, 2025. It is now read-only.

Overwatch test 1 #1204

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 7 additions & 2 deletions .github/workflows/upload-overwatch.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,14 @@
name: Upload Overwatch
# Created on 2025-03-12 09:39:33
name: Upload Overwatch - 2025-03-12 09:39:33

on:
pull_request:
types:
- opened
- synchronize
push:
branches-ignore:
- main

jobs:
upload-overwatch:
Expand Down Expand Up @@ -39,5 +43,6 @@ jobs:
./overwatch-cli \
--auth-token ${{ secrets.SENTRY_AUTH_TOKEN }} \
--organization-slug codecov \
--pullid 1204 \
python \
--python-path $(which python3)
--python-path $(which python3)
3 changes: 0 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -63,14 +63,11 @@ lint.install.local:
uv add --dev ruff

lint.run:
ruff check
ruff format

lint.check:
echo "Linting..."
ruff check
echo "Formatting..."
ruff format --check

build.requirements:
# if docker pull succeeds, we have already build this version of
Expand Down
8 changes: 8 additions & 0 deletions services/components.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,14 @@ def component_filtered_report(
"""
Filter a report such that the totals, etc. are only pertaining to the given component.
"""
print(report)
print(components)

# This will fail mypy with "Item "None" of "Optional[Report]" has no attribute "filter""
# Because we're not checking if report is None before accessing filter()
report: Optional[Report] = report
filtered_report = report.filter(flags=flags, paths=paths)

flags, paths = [], []
for component in components:
flags.extend(component.get_matching_flags(report.get_flag_names()))
Expand Down
Loading