Skip to content

Fix some runtime warnings#220

Merged
splittingred merged 2 commits intobigcommerce:mainfrom
ydah:fix-warning
Jan 29, 2026
Merged

Fix some runtime warnings#220
splittingred merged 2 commits intobigcommerce:mainfrom
ydah:fix-warning

Conversation

@ydah
Copy link
Contributor

@ydah ydah commented Dec 24, 2024

What? Why?

A runtime warning was output when executing in the following environment, which has been corrected:

❯ ruby -v
ruby 3.4.0rc1 (2024-12-12 master 29caae9991) +PRISM [arm64-darwin23]
❯ RUBYOPT=-w bundle exec rake
/ydah/gruf/spec/support/helpers.rb:97: warning: possibly useless use of nil in void context
/ydah/gruf/spec/support/helpers.rb:88: warning: assigned but unused variable - e

How was it tested?

Verified that existing tests were not broken.

@splittingred splittingred merged commit d0112c2 into bigcommerce:main Jan 29, 2026
20 checks passed
@ydah ydah deleted the fix-warning branch January 29, 2026 22:07
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