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

Fix: path to coverage report #1974

Merged
merged 1 commit into from
Mar 22, 2024
Merged

Fix: path to coverage report #1974

merged 1 commit into from
Mar 22, 2024

Conversation

angela-tran
Copy link
Member

Fixing a 404 I noticed on our docs site:

The mkdocs.yml GitHub workflow publishes the coverage report at docs/tests/coverage.

The link to the coverage report was on a page that got moved under docs/tests/automated-tests in a5c63c5, and we forgot to update the link.

the `mkdocs.yml` GitHub workflow publishes it at `docs/tests/coverage`
@angela-tran angela-tran self-assigned this Mar 22, 2024
@angela-tran angela-tran requested a review from a team as a code owner March 22, 2024 17:50
@github-actions github-actions bot added the documentation [auto] Improvements or additions to documentation label Mar 22, 2024
Copy link

Coverage report

This PR does not seem to contain any modification to coverable code.

Copy link

Copy link
Member

@thekaveman thekaveman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. It's annoying we can't test this in the preview site but meh

@angela-tran angela-tran merged commit 0fa3235 into dev Mar 22, 2024
12 checks passed
@angela-tran angela-tran deleted the fix/coverage-report-path branch March 22, 2024 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation [auto] Improvements or additions to documentation
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants