Skip to content

Drop unused variable#6

Merged
olleolleolle merged 1 commit intosocketry:mainfrom
wtn:time
Mar 16, 2026
Merged

Drop unused variable#6
olleolleolle merged 1 commit intosocketry:mainfrom
wtn:time

Conversation

@wtn
Copy link
Contributor

@wtn wtn commented Mar 16, 2026

When running tests in a project that uses this gem, I encountered this warning:

/lib/ruby/gems/3.3.0/gems/async-limiter-2.0.0/lib/async/limiter/timing/fixed_window.rb:28: warning: assigned but unused variable - current_time

Types of Changes

  • Maintenance.

Contribution

Copy link

@olleolleolle olleolleolle left a comment

Choose a reason for hiding this comment

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

Looks reasonable!

@olleolleolle olleolleolle merged commit acd525e into socketry:main Mar 16, 2026
17 of 20 checks passed
@wtn wtn deleted the time branch March 16, 2026 21:05
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