Skip to content

Fix rubocop setup (for some envs)#2377

Merged
solnic merged 5 commits intomasterfrom
solnic/fix-rubocop-setup
Aug 16, 2024
Merged

Fix rubocop setup (for some envs)#2377
solnic merged 5 commits intomasterfrom
solnic/fix-rubocop-setup

Conversation

@solnic
Copy link
Copy Markdown
Collaborator

@solnic solnic commented Aug 16, 2024

This makes it possible to use rubocop when working from inside a subdir.
It also fixes running rubocop from the root where it would actually use
the config that we have. Previously it would ignore it for some reason,
symlinking did the trick.

The rubocop-packaging was needed because something somewhere tries to
require it.

#skip-changelog

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 16, 2024

Codecov Report

Attention: Patch coverage is 93.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 98.70%. Comparing base (ec70677) to head (08c3089).
Report is 1 commits behind head on master.

Files Patch % Lines
sentry-resque/lib/sentry/resque.rb 66.66% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #2377       +/-   ##
===========================================
+ Coverage   78.10%   98.70%   +20.60%     
===========================================
  Files         149      209       +60     
  Lines        7011    13861     +6850     
===========================================
+ Hits         5476    13682     +8206     
+ Misses       1535      179     -1356     
Components Coverage Δ
sentry-ruby 99.06% <ø> (+43.94%) ⬆️
sentry-rails 97.43% <100.00%> (ø)
sentry-sidekiq 97.01% <100.00%> (ø)
sentry-resque 97.11% <66.66%> (ø)
sentry-delayed_job 98.92% <ø> (ø)
sentry-opentelemetry 100.00% <100.00%> (ø)
Files Coverage Δ
...entelemetry/lib/sentry/opentelemetry/propagator.rb 100.00% <100.00%> (ø)
...lemetry/lib/sentry/opentelemetry/span_processor.rb 100.00% <100.00%> (ø)
...entry-rails/lib/sentry/rails/capture_exceptions.rb 96.66% <100.00%> (ø)
sentry-rails/lib/sentry/rails/configuration.rb 97.43% <100.00%> (ø)
...y-rails/lib/sentry/rails/controller_transaction.rb 95.45% <100.00%> (ø)
sentry-rails/lib/sentry/rails/railtie.rb 98.75% <100.00%> (ø)
sentry-sidekiq/lib/sentry/sidekiq/error_handler.rb 93.10% <100.00%> (ø)
...iq/lib/sentry/sidekiq/sentry_context_middleware.rb 97.56% <100.00%> (ø)
sentry-resque/lib/sentry/resque.rb 97.56% <66.66%> (ø)

... and 128 files with indirect coverage changes

This makes it possible to use rubocop when working from inside a subdir.
It also fixes running rubocop from the root where it would actually use
the config that we have. Previously it would ignore it for some reason,
symlinking did the trick.

The rubocop-packaging was needed because something somewhere tries to
require it.
@solnic solnic force-pushed the solnic/fix-rubocop-setup branch from d85c58c to be3764a Compare August 16, 2024 15:54
@solnic solnic marked this pull request as ready for review August 16, 2024 16:02
@solnic solnic requested review from sl0thentr0py and st0012 and removed request for st0012 August 16, 2024 16:02
Copy link
Copy Markdown
Contributor

@st0012 st0012 left a comment

Choose a reason for hiding this comment

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

🙌

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.

2 participants