Skip to content

feat: improve sentry reports #3190

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

Merged
merged 1 commit into from
Jun 9, 2023

Conversation

Codelax
Copy link
Member

@Codelax Codelax commented Jun 9, 2023

Switch to the new sentry sdk. The stack trace is cleaned up.
Added some informations to the report:

  • The command used
  • The list of arguments used without their content
  • The CLI version as release

@codecov-commenter
Copy link

Codecov Report

Merging #3190 (4cb7735) into master (8c177ff) will decrease coverage by 0.02%.
The diff coverage is 26.66%.

@@            Coverage Diff             @@
##           master    #3190      +/-   ##
==========================================
- Coverage   75.64%   75.62%   -0.02%     
==========================================
  Files         177      177              
  Lines       38224    38238      +14     
==========================================
+ Hits        28915    28919       +4     
- Misses       8243     8253      +10     
  Partials     1066     1066              
Impacted Files Coverage Δ
cmd/scw/main.go 0.00% <0.00%> (ø)
internal/core/build_info.go 50.76% <0.00%> (-6.13%) ⬇️
internal/core/shell.go 8.63% <0.00%> (-0.12%) ⬇️
internal/core/cobra_utils.go 65.81% <100.00%> (+0.71%) ⬆️

@remyleone remyleone enabled auto-merge June 9, 2023 13:57
@remyleone remyleone added this pull request to the merge queue Jun 9, 2023
Merged via the queue into scaleway:master with commit d7705c3 Jun 9, 2023
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.

3 participants