Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SendContext fix #300

Merged
merged 2 commits into from
Mar 24, 2021
Merged

SendContext fix #300

merged 2 commits into from
Mar 24, 2021

Commits on Mar 23, 2021

  1. SendContext didn't check if context is done first. Instead, in case c… 

    …tx is done and channel to send has a free buffer, it chose randomly either to return false or send value to the channel.

Added a test that failed randomly on previous version of SendContext.

Changes to be committed:
	modified:   item.go
	modified:   item_test.go
    @pashaosipyants
    pashaosipyants committed Mar 23, 2021
    Configuration menu
    Copy the full SHA
    d22820a View commit details
    Browse the repository at this point in the history

Commits on Mar 24, 2021

  1. Update item.go 

    Co-authored-by: Teiva Harsanyi <[email protected]>
    @pashaosipyants @teivah
    pashaosipyants and teivah authored Mar 24, 2021
    Configuration menu
    Copy the full SHA
    d4af109 View commit details
    Browse the repository at this point in the history