Question about the override to Rake::Task#execute

[hosamaly]

I'm a bit confused about this piece of code:

sentry-ruby/sentry-ruby/lib/sentry/rake.rb

Lines 21 to 28 in f15f58e

	module Task
	# @api private
	def execute(args=nil)
	return super unless Sentry.initialized? && Sentry.get_current_hub
	super
	end
	end

The override doesn't seem to do much other than call Sentry.get_current_hub.

If this is the case, it would be slightly clearer, in my humble opinion, to define it as:

def execute(args=nil) 
  Sentry.get_current_hub if Sentry.initialized?
  super
end

Did I understand the purpose of the method correctly?

[sl0thentr0py]

I think this is leftover code from #1617, I don't even think we need the Task patch anymore. @st0012 can you verify?

[st0012]

@sl0thentr0py Yes you're right. I've opened #2214 to remove it.
@hosamaly Thanks for raising this!