search

rspec / rspec-expectations

Provides a readable API to express expected outcomes of a code example
https://rspec.info
MIT License
1.25k stars 399 forks source link

Use `RSpec.warning` instead of `::Kernel.warn` #1461

Open runephilosof-abtion opened 2 weeks ago

runephilosof-abtion commented 2 weeks ago

I expected this to raise an error when Rspec.configure { |config| config.raise_on_warning = true }

https://github.com/rspec/rspec-expectations/blob/0139c22a62624849c09dd33980554215e63ab9eb/lib/rspec/expectations/handler.rb#L7-L11

pirj commented 2 weeks ago

Nice catch, @runephilosof-abtion . A PR is welcome.