Upload backend coverage report on failure

[ejfine]

Why is this change necessary?

Trying to debug CI coverage failures

How does this change address the issue?

Adds step to upload HTML coverage report if failure

What side effects does this change have?

None

How is this change tested?

Downstream repo

Other

Ignoring changes to favicon and vendor files

[Copilot]

Pull Request Overview

Adds logic to upload the backend coverage report when CI fails, updates lint exclusions for vendor files, and prevents favicon from being overwritten by the template.

• Excludes vendor_files/ directories from pre-commit checks.
• Adjusts the backend unit-test step and adds a failure-only artifact upload.
• Adds a copier directive to skip overwriting an existing favicon.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
template/.pre-commit-config.yaml.jinja	Exclude vendor_files/ from formatting and snapshot checks
template/.github/workflows/ci.yaml.jinja	Remove XML report, add HTML report upload step on failure
copier.yml	Add _skip_if_exists for favicon to avoid overwriting

Comments suppressed due to low confidence (3)

template/.github/workflows/ci.yaml.jinja:65

• The leading hyphen on the run: line creates a new unnamed step and breaks the YAML structure. It should be indented under the - name: Unit test step without an extra dash.

-        run: uv --directory=backend run pytest tests/unit --durations=5

template/.pre-commit-config.yaml.jinja:64

• [nitpick] The same vendor_files exclusion appears in multiple repos; consider consolidating into a shared variable or block to reduce duplication.

.*\/vendor_files\/.*|

copier.yml:178

• Copier’s standard key is skip_if_exists (without a leading underscore). Using _skip_if_exists may not have any effect—rename the key or confirm the correct directive.

_skip_if_exists:

[Copilot]

The coverage report is no longer generated since --cov-report=xml was removed; the upload step will have no files to send. Consider adding --cov-report=html:backend/coverage-report-pytest (or a similar option) to generate the directory you intend to upload.

Suggested change

	run: uv --directory=backend run pytest tests/unit --durations=5
	run: uv --directory=backend run pytest tests/unit --durations=5 --cov=backend --cov-report=html:backend/coverage-report-pytest