Skip to content

Guide users towards overwriting files - #990

Merged
godlygeek merged 1 commit into
bloomberg:mainfrom
godlygeek:hint_users_towards_overwriting
Aug 7, 2026
Merged

Guide users towards overwriting files#990
godlygeek merged 1 commit into
bloomberg:mainfrom
godlygeek:hint_users_towards_overwriting

Conversation

@godlygeek

Copy link
Copy Markdown
Contributor

Users may not know that the --force option exists (or its equivalent overwrite=True API argument for memray.Tracker). Try to point them to it in our error messages to help them out.

Closes #828

@godlygeek godlygeek self-assigned this Aug 7, 2026
@godlygeek
godlygeek force-pushed the hint_users_towards_overwriting branch from 0f3a767 to 6a936d4 Compare August 7, 2026 17:37
@godlygeek
godlygeek force-pushed the hint_users_towards_overwriting branch from 6a936d4 to 14732db Compare August 7, 2026 18:02
Users may not know that the `--force` option exists (or its equivalent
`overwrite=True` API argument for `memray.Tracker`). Try to point them
to it in our error messages to help them out.

Signed-off-by: Matt Wozniski <mwozniski@bloomberg.net>
@godlygeek
godlygeek force-pushed the hint_users_towards_overwriting branch from 14732db to 64d74a0 Compare August 7, 2026 18:05
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.47%. Comparing base (d6787da) to head (64d74a0).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #990      +/-   ##
==========================================
+ Coverage   92.43%   92.47%   +0.04%     
==========================================
  Files         101      101              
  Lines       13199    13211      +12     
  Branches      477      477              
==========================================
+ Hits        12200    12217      +17     
+ Misses        999      994       -5     
Flag Coverage Δ
cpp 92.47% <100.00%> (+0.04%) ⬆️
python_and_cython 92.47% <100.00%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@sarahmonod sarahmonod 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.

LGTM

@godlygeek
godlygeek merged commit 019e1a9 into bloomberg:main Aug 7, 2026
24 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Overwriting output files if they already exist

3 participants