Skip to content

chore: stop tracking test-run artifacts - #393

Open
mhovd wants to merge 1 commit into
mainfrom
mhovd-untrack-test-artifacts
Open

mhovd wants to merge 1 commit into
mainfrom
mhovd-untrack-test-artifacts

Conversation

@mhovd

@mhovd mhovd commented Sep 16, 2026 •

Copy link
Copy Markdown
Collaborator

Untracks three files that are outputs of running the test suite

  • tests/testthat/Rplots.pdf
  • tests/testthat/test.png
  • tests/testthat/testthat-problems.rds (427 KB of stale failures from an old run)

Rplots.pdf, test.png and testthat-problems.rds are outputs of running the
test suite, not inputs: nothing in the package, tests, or CI reads them.
testthat-problems.rds held 25 stale failures from an old run, and testthat
deletes or rewrites it, so the working tree came back dirty after a suite
run and the file had to be restored by hand.

Untracked with 'git rm --cached', so the files stay on disk, and ignored
by name so they cannot be re-added from any directory.
Copilot AI lite review requested due to automatic review settings September 16, 2026 11:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.77%. Comparing base (ef474a1) to head (a137f96).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #393   +/-   ##
=======================================
  Coverage   64.77%   64.77%           
=======================================
  Files          52       52           
  Lines       22964    22964           
=======================================
  Hits        14876    14876           
  Misses       8088     8088           
🚀 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.

This branch has not been deployed

No deployments
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.

3 participants