Skip to content

Add deprecation notices - #811

Merged
trunk-io[bot] merged 3 commits into
mainfrom
gabe/deprecate-notices
Nov 6, 2025
Merged

trunk-io[bot] merged 3 commits into
mainfrom
gabe/deprecate-notices

Conversation

@gnalh

@gnalh gnalh commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

We have a few flags that have been deprecated for a while. Let's add notices that will log in sentry.

@trunk-io

trunk-io Bot commented Nov 5, 2025 •

Copy link
Copy Markdown

😎 Merged successfully - details.

@codecov-commenter

codecov-commenter commented Nov 5, 2025 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 35.71429% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.10%. Comparing base (99314a0) to head (e192aca).

Files with missing lines Patch % Lines
cli/src/upload_command.rs 50.00% 5 Missing ⚠️
cli/src/validate_command.rs 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #811      +/-   ##
==========================================
+ Coverage   72.86%   73.10%   +0.24%     
==========================================
  Files          72       72              
  Lines       16024    16038      +14     
==========================================
+ Hits        11676    11725      +49     
+ Misses       4348     4313      -35     

☔ View full report in Codecov by Sentry.
📢 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.

@gnalh
gnalh changed the base branch from gabe/trunk-16938-update-cli-descriptions to main November 5, 2025 23:45
@trunk-io
trunk-io Bot merged commit e3dc742 into main Nov 6, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants